remove dead stringstream stuff

RESOLVED FIXED in mozilla1.1beta

Status

()

P2
normal
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: alecf, Assigned: alecf)

Tracking

Trunk
mozilla1.1beta
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fix in hand)

Attachments

(1 attachment)

(Assignee)

Description

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

17 years ago
Created attachment 89174 [details] [diff] [review]
remove stringstream junk

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

17 years ago
Comment on attachment 89174 [details] [diff] [review]
remove stringstream junk

check the commercial tree too?

r=dougt
Attachment #89174 - Flags: review+
Comment on attachment 89174 [details] [diff] [review]
remove stringstream junk

sr=dveditz
Attachment #89174 - Flags: superreview+
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: fix in hand
Target Milestone: --- → mozilla1.1beta
(Assignee)

Comment 4

17 years ago
cool..just checked commercial, we're all clear.
fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Updated

17 years ago
Blocks: 63025
You need to log in before you can comment on or make changes to this bug.