Closed
Bug 70142
Opened 24 years ago
Closed 24 years ago
combine |nsAReadableString| and |nsAWritableString| to avoid multiple inheritance
Categories
(Core :: XPCOM, defect, P1)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla0.9
People
(Reporter: scc, Assigned: scc)
References
Details
keep the names as typedefs; but combine the two interfaces into one. A |const|
instance has exactly the capabilities as an |nsAReadableString|, a non-|const|
instance is a |nsAWritableString|. This is needed so that, e.g., I don't have
to have both a readable and writable |nsAFlatString|, and can avoid multiple
inheritance down the line.
Assignee | ||
Updated•24 years ago
|
Assignee | ||
Comment 1•24 years ago
|
||
How about |nsAString|? That fits with the rest of the names.
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9 → mozilla0.8.1
Assignee | ||
Comment 2•24 years ago
|
||
if I have to build a new file "nsAString.h" anyway, I might as well pull out all
the pieces from "nsAReadableString.h" and "nsAWritableString.h" into the
appropriate files at the same time. Now would be the convenient time to
un-templatize them as well. So, making this bug depend on 70077 and 70078,
though I'll probably attach the patches here.
Assignee | ||
Comment 3•24 years ago
|
||
this has been checked in on the STRING_081_BRANCH; when the branch is merged
into the trunk, I'll close this bug
Assignee | ||
Comment 4•24 years ago
|
||
(mass change) didn't get these in for target milestone mozilla0.8.1 but they are
very close. Moving all to mozilla0.9.
Target Milestone: mozilla0.8.1 → mozilla0.9
Assignee | ||
Comment 5•24 years ago
|
||
landed the branch; fix checked in
Assignee | ||
Comment 6•24 years ago
|
||
and marking fixed
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Updated•4 years ago
|
Component: String → XPCOM
You need to log in
before you can comment on or make changes to this bug.
Description
•