Closed
Bug 230599
Opened 21 years ago
Closed 21 years ago
remove unused nsStatistics.*
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
People
(Reporter: bryner, Assigned: bryner)
Details
Attachments
(1 file)
5.76 KB,
patch
|
dougt
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
We're not using this class for anything. It can go away.
Assignee | ||
Comment 1•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #138780 -
Flags: superreview?(darin)
Attachment #138780 -
Flags: review?(dougt)
Comment 2•21 years ago
|
||
Comment on attachment 138780 [details] [diff] [review] patch i would rather just ifdef DEBUG this. thoughts?
Attachment #138780 -
Flags: review?(dougt) → review+
Assignee | ||
Comment 3•21 years ago
|
||
I'd rather not unless we have #ifdef DEBUG code that makes use of it. It's another 2 files to worry about keeping up to date with API changes, license changes, etc.
Assignee | ||
Comment 4•21 years ago
|
||
Comment on attachment 138780 [details] [diff] [review] patch darin's out of town... jst, can you sr?
Attachment #138780 -
Flags: superreview?(darin) → superreview?(jst)
Comment 5•21 years ago
|
||
Comment on attachment 138780 [details] [diff] [review] patch sr=jst
Attachment #138780 -
Flags: superreview?(jst) → superreview+
Assignee | ||
Comment 6•21 years ago
|
||
checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•