Closed Bug 822263 Opened 12 years ago Closed 12 years ago

Change nsAbView.cpp to use DebugOnly.h instead of Util.h

Categories

(MailNews Core :: Address Book, defect)

defect
Not set
blocker

Tracking

(seamonkey2.17 fixed)

VERIFIED FIXED
Thunderbird 20.0
Tracking Status
seamonkey2.17 --- fixed

People

(Reporter: mconley, Assigned: iannbugzilla)

References

Details

Attachments

(1 file)

Bug 820570 moved DebugOnly from Util.h into DebugOnly.h.

This is causing build-breakage for us. We simply need to switch us over to using DebugOnly.h.
This uses the same format as the m-c patch, so puts the new include at the beginning of the file.
Currently testing locally and then will push to try.
Assignee: nobody → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #692965 - Flags: review?(mconley)
Moving to MailNews Core.
Severity: normal → blocker
Product: Thunderbird → MailNews Core
Hardware: x86 → All
Version: 20 → Trunk
Comment on attachment 692965 [details] [diff] [review]
Switch from Util.h to DebugOnly.h [Checked in: Comment 7]

Review of attachment 692965 [details] [diff] [review]:
-----------------------------------------------------------------

Perfect. Thanks Ian!
Attachment #692965 - Flags: review?(mconley) → review+
FWIW, this patch fixed my busted build.
Do we need a mailnews peer review here?
Keywords: checkin-needed
I think it just needs to land. I'll do it.
Landed on comm-central as https://hg.mozilla.org/comm-central/rev/8b55bdd4b0a3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 20.0
https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=3a140ce538e5
No bustage and fails are already known.
Attachment #692965 - Attachment description: Switch from Util.h to DebugOnly.h → Switch from Util.h to DebugOnly.h [Checked in: Comment 7]
Worked for me on Linux.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: