Closed Bug 46892 Opened 24 years ago Closed 15 years ago

make |nsStr| and other obsolete string implementation go away

Categories

(Core :: XPCOM, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: dougt, Assigned: jag+mozilla)

References

Details

Attachments

(1 file)

When looking at nsStr::Destroy(), (mCapacity - mLength), on the average, is 54 bytes.
Note, though, that |nsStr| as a class is going away altogether. Strings are soon to be allocated at exactly the correct size and grown for operations that fault on space. User calls to set the capacity will be honored exactly.
Status: NEW → ASSIGNED
if that is the case, this bug will become invalid
This would be a lot more useful if it disregarded cases where the nsStr is part of an AutoString. The waste there is just not a problem.
Updating QA Contact
QA Contact: leger → dougt
nsStr is soon to go away entirely; I'll mark this bug fixed at that time
Component: XPCOM → String
OS: Windows NT → All
Hardware: PC → All
Summary: nsStr wastes on the average of 54 bytes → make |nsStr| go away
Target Milestone: --- → mozilla0.9.1
Blocks: 28221
No longer blocks: 28221
Depends on: 53065, 69872
Summary: make |nsStr| go away → make |nsStr| and other obsolete string implementation go away
No longer depends on: 53065, 69872
Depends on: 53065, 69872
Target Milestone: mozilla0.9.1 → mozilla0.9
Target Milestone: mozilla0.9 → mozilla1.1
*** Bug 46738 has been marked as a duplicate of this bug. ***
re-targeting milestones, starting from a clean slate
Target Milestone: mozilla1.1 → ---
giving up ancient string bugs to the new string owner. jag, you'll want to sort through these and see which ones still apply and go with or against the direction in which you intend strings evolve
Assignee: scc → jaggernaut
Status: ASSIGNED → NEW
QA Contact: doug.turner → string
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WORKSFORME
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: