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

RESOLVED FIXED in mozilla0.9

Status

()

Core
String
P1
normal
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: Scott Collins, Assigned: Scott Collins)

Tracking

Trunk
mozilla0.9
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

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

17 years ago
Blocks: 70075
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9
(Assignee)

Comment 1

17 years ago
How about |nsAString|?  That fits with the rest of the names.
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9 → mozilla0.8.1
(Assignee)

Comment 2

17 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.
Depends on: 70077, 70078
(Assignee)

Comment 3

17 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

17 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)

Updated

17 years ago
Blocks: 73786
(Assignee)

Comment 5

17 years ago
landed the branch; fix checked in
(Assignee)

Comment 6

17 years ago
and marking fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.