Closed
Bug 1087409
Opened 10 years ago
Closed 10 years ago
FORCE_PR_LOG is now globally defined
Categories
(MailNews Core :: Backend, defect)
Tracking
(thunderbird36 fixed)
RESOLVED
FIXED
Thunderbird 36.0
| Tracking | Status | |
|---|---|---|
| thunderbird36 | --- | fixed |
People
(Reporter: jcranmer, Assigned: jcranmer)
Details
Attachments
(1 file, 1 obsolete file)
|
23.79 KB,
patch
|
Irving
:
review+
|
Details | Diff | Splinter Review |
This cuts down on a little bit of warning spam.
Attachment #8509559 -
Flags: review?(irving)
|
||
Comment 1•10 years ago
|
||
Comment on attachment 8509559 [details] [diff] [review] Remove redundant FORCE_PR_LOG lines Review of attachment 8509559 [details] [diff] [review]: ----------------------------------------------------------------- Also, check if MOZ_LOGGING is unused after all these removals. ::: ldap/xpcom/src/nsLDAPInternal.h @@ +8,1 @@ > #endif Remove the empty #ifdef MOZ_LOGGING ::: mailnews/base/src/nsMsgBiffManager.cpp @@ +7,1 @@ > #endif here ::: mailnews/base/src/nsMsgCopyService.cpp @@ +7,1 @@ > #endif there ::: mailnews/base/src/nsMsgPurgeService.cpp @@ +8,1 @@ > #endif everywhere
Attachment #8509559 -
Flags: review?(irving) → review-
|
Assignee | |
Comment 2•10 years ago
|
||
Now with fewer empty #ifdefs.
Attachment #8509559 -
Attachment is obsolete: true
Attachment #8510325 -
Flags: review?(irving)
|
|
||
Updated•10 years ago
|
Attachment #8510325 -
Flags: review?(irving) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
https://hg.mozilla.org/comm-central/rev/4b11354bfcf1
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 36.0
|
||
Updated•9 years ago
|
status-thunderbird36:
--- → fixed
|
|
Assignee | |
Updated•9 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•