Closed Bug 70142 Opened 24 years ago Closed 23 years ago

combine |nsAReadableString| and |nsAWritableString| to avoid multiple inheritance

Categories

(Core :: XPCOM, defect, P1)

Product:
Core
Component:
XPCOM
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
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.
Blocks: 70075
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9
How about |nsAString|?  That fits with the rest of the names.
Target Milestone: mozilla0.9 → mozilla0.8.1
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.
Depends on: 70077, 70078
this has been checked in on the STRING_081_BRANCH; when the branch is merged
into the trunk, I'll close this bug

(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
Blocks: 73786
landed the branch; fix checked in
and marking fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.