Closed Bug 117927 Opened 23 years ago Closed 23 years ago

remove unused static variables in editor, netwerk, layout, content

Categories

(Core :: DOM: Editor, defect)

PowerPC
Mac System 9.x
defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla0.9.8

People

(Reporter: Brade, Assigned: Brade)

Details

Attachments

(1 file)

This bug is to cover removal of unused static variables in mozilla/editor r=glazou; rs=shaver
272 bytes removed from data today; more to come on another day
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
darin has given me an rs= for mozilla/netwerk
Summary: remove unused static variables in editor → remove unused static variables in editor, netwerk, layout, content
A quick first pass of netwerk files found 345 bytes of unused static variables or constants. They are now checked in.
I'm not sure what you're using to find these, but one easy way to find them is to look at the warnings given by the IRIX native compiler. Problem is, the one on tinderbox (cement on Seamonkey-Ports) is a depend build, so you need to get a good log after someone causes a full rebuild. (However, the IRIX compiler won't warn about ones that aren't declared |static|.)
A quick pass at mozilla/layout found 465 bytes which have now been removed. dbaron--I am using the link map file on Mac (produced with CodeWarrior).
I have 643 bytes of data that can be removed in mozilla/content...
Comment on attachment 64168 [details] [diff] [review] removal of unused static/globals in mozilla/content Nice, sr=jst
Attachment #64168 - Flags: superreview+
Comment on attachment 64168 [details] [diff] [review] removal of unused static/globals in mozilla/content r/sr=darin
Attachment #64168 - Flags: review+
resolving this bug as fixed; there are more to be removed but they get slightly trickier (they'll have to wait for another bug)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Kathy, please verify this one...thanks.
v
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: