Closed Bug 577910 Opened 10 years ago Closed 9 years ago
mark DEBUG only variables as ifdef DEBUG in jsd
this is part of a crusade to get rid of compilation warnings
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #456737 - Flags: review?(cbiesinger)
10 years ago
Attachment #456737 - Flags: review?(cbiesinger) → review+
I'm not really a jsd peer though...
Summary: use NS_DEBUG_ASSIGN for jsd → mark DEBUG only variables as ifdef DEBUG in jsd
Attachment #463497 - Flags: approval2.0? → approval2.0+
Comment on attachment 463497 [details] [diff] [review] using #ifdef DEBUG Removing approval - this has been sitting since August. Renominate if someone can show me that this hasn't bitrotten and is still needed.
Attachment #463497 - Flags: approval2.0+
beltzner: i'm the module owner. it applied as of the last time i did a qrefresh to my queue: http://hg.mozilla.org/users/timeless_mozdev.org/mozilla-central-mq/file/default/ i'm not quite sure what proof you want. but give me an airline ticket to your location and i'll gladly show you whatever proof you request.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.