Closed Bug 243109 Opened 20 years ago Closed 19 years ago

cleanup business of internal vs. external definition of nsAString

Categories

(Core :: XPCOM, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla1.8beta3

People

(Reporter: darin.moz, Assigned: benjamin)

Details

cleanup business of internal vs. external definition of nsAString

see bug 205425 comment 67 through bug 205425 comment 74.
targeting 1.8a, but whatever solution we come up with should probably land on
the 1.7 branch at some point.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.8alpha
Blocks: 205425
Target Milestone: mozilla1.8alpha1 → mozilla1.8beta
i think we can add some #define sugar in nsStringFwd.h to map all of our string
class names to something with a _internal suffix or some such mangling.  but to
make that work, we'd also need to add the same mangling in nsrootidl.idl for
nsAString and nsACString (or maybe just #include nsStringFwd.h there).

solving this bug will make it possible to mix the API defined in nsStringAPI.h
with that of the internal string API all within the same module.  that would
make it possible or easier to support static linking of mfcembed again (see bug
 262218).
No longer blocks: 205425
I may not have time to implement this for Gecko 1.8.  Help wanted.
Severity: normal → minor
Keywords: helpwanted
Priority: -- → P4
Target Milestone: mozilla1.8beta1 → mozilla1.8beta2
I need this in order for embedders to be able to use the frozen string API on
linux/mac.
Assignee: darin → benjamin
Severity: minor → critical
Status: ASSIGNED → NEW
Flags: blocking1.8b4+
Keywords: helpwanted
Priority: P4 → P1
Target Milestone: mozilla1.8beta2 → mozilla1.8beta3
Fixed by bug 297923.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.