Closed Bug 70142 Opened 24 years ago Closed 24 years ago

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

Categories

(Core :: XPCOM, defect, P1)

defect

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.
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: 24 years ago
Resolution: --- → FIXED
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.