Closed Bug 794558 Opened 7 years ago Closed 7 years ago
Remove 2 unused vars in mailnews/base/src
/Users/stefan/Moz/comm-central/mailnews/base/src/nsMessengerOSXIntegration.mm:66:25: warning: unused variable 'MsgDockCountsLogModule' [-Wunused-variable] static PRLogModuleInfo *MsgDockCountsLogModule = nullptr; /Users/stefan/Moz/comm-central/mailnews/base/src/nsMsgStatusFeedback.cpp:33:12: warning: unused variable 'rv' [-Wunused-variable] nsresult rv; Re nsMessengerOSXIntegration.mm: Seems you added logging in http://hg.mozilla.org/comm-central/rev/2acf5df83607 which wasn't 100% removed later on (http://hg.mozilla.org/comm-central/rev/53f0b1dc8aab).
Attachment #665052 - Flags: review?(mozilla)
Comment on attachment 665052 [details] [diff] [review] Remove unused vars (and remnants of logging in nsMessengerOSXIntegration.mm) Looks great, thanks.
Attachment #665052 - Flags: review?(mozilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
You need to log in before you can comment on or make changes to this bug.