cleanup business of internal vs. external definition of nsAString

RESOLVED FIXED in mozilla1.8beta3

Status

()

Core
String
P1
critical
RESOLVED FIXED
14 years ago
12 years ago

People

(Reporter: Darin Fisher, Assigned: Benjamin Smedberg)

Tracking

Trunk
mozilla1.8beta3
Points:
---
Bug Flags:
blocking1.8b5 +

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

14 years ago
cleanup business of internal vs. external definition of nsAString

see bug 205425 comment 67 through bug 205425 comment 74.
(Reporter)

Comment 1

14 years ago
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

Updated

13 years ago
Blocks: 205425
(Reporter)

Updated

13 years ago
Target Milestone: mozilla1.8alpha1 → mozilla1.8beta
(Reporter)

Comment 2

13 years ago
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).

Updated

13 years ago
No longer blocks: 205425
(Reporter)

Comment 3

13 years ago
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
(Assignee)

Comment 4

12 years ago
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
(Assignee)

Comment 5

12 years ago
Fixed by bug 297923.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.