Closed Bug 1761700 Opened 2 years ago Closed 2 years ago

Remove MOZ_STRING_WITH_OBSOLETE_API

Categories

(Core :: String, task)

task

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: emilio, Assigned: emilio)

Details

Attachments

(1 file)

I was shuffling some files today (bug 1761693) and hit some interesting build errors because of this. The setup is: a.cpp includes nsTString.h (which doesn't define this), b.cpp includes nsString.h, which does, and uses some of the "obsolete" APIs.

Depending on the order of a.cpp and b.cpp in the unified build, the build works or doesn't, and the error message is a bit cryptic (b is including the right file after all).

Nobody explicitly disables it, but we have some users that include just
nsTString.h and can cause issues.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/094420cf0c4d
Remove MOZ_STRING_WITH_OBSOLETE_API. r=xpcom-reviewers,nika
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: