Closed
Bug 154265
Opened 22 years ago
Closed 22 years ago
remove dead stringstream stuff
Categories
(Core :: XPCOM, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla1.1beta
People
(Reporter: alecf, Assigned: alecf)
References
Details
(Whiteboard: fix in hand)
Attachments
(1 file)
|
17.54 KB,
patch
|
dougt
:
review+
dveditz
:
superreview+
|
Details | Diff | Splinter Review |
There is a bunch of outdated, unused code in xpcom/io mostly consisting of stringstream stuff - it uses some old-sk00l stream classes and isn't maintained. we should just nuke it. patch forthcoming.
|
Assignee | |
Comment 1•22 years ago
|
||
This removes as much as I could without impacting existing consumers. Removes two classes that were unnecessary, as well as a bunch of exported functions. Double-checked with lxr to make sure nobody was using these.
|
||
Comment 2•22 years ago
|
||
Comment on attachment 89174 [details] [diff] [review] remove stringstream junk check the commercial tree too? r=dougt
Attachment #89174 -
Flags: review+
|
||
Comment 3•22 years ago
|
||
Comment on attachment 89174 [details] [diff] [review] remove stringstream junk sr=dveditz
Attachment #89174 -
Flags: superreview+
|
|
Assignee | |
Updated•22 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: fix in hand
Target Milestone: --- → mozilla1.1beta
|
|
Assignee | |
Comment 4•22 years ago
|
||
cool..just checked commercial, we're all clear. fixed.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•