Closed Bug 1146094 Opened 6 years ago Closed 6 years ago

Stop using MOZ_OVERRIDE and MOZ_FINAL in comm-central. Instead use the override and final keywords.

Categories

(MailNews Core :: Backend, defect)

defect
Not set
normal

Tracking

(thunderbird38 fixed)

RESOLVED FIXED
Thunderbird 39.0
Tracking Status
thunderbird38 --- fixed

People

(Reporter: philip.chee, Assigned: ehsan)

References

Details

Attachments

(1 obsolete file)

(In Bug #1145631 comment #0)
> We can now use the override and final keywords.

(In Bug #1145631 comment #2)

> This patch was automatically generated using the following script:
> 
> 
> function convert() {
> echo "Converting $1 to $2..."
> find . \
>        ! -wholename "*/.git*" \
>        ! -wholename "obj-ff-dbg*" \
>          -type f \
>       \( -iname "*.cpp" \
>          -o -iname "*.h" \
>          -o -iname "*.c" \
>          -o -iname "*.cc" \
>          -o -iname "*.idl" \
>          -o -iname "*.ipdl" \
>          -o -iname "*.ipdlh" \
>          -o -iname "*.mm" \) | \
>     xargs -n 1 sed -i -e "s/\b$1\b/$2/g"
> }
> 
> convert MOZ_OVERRIDE override
> convert MOZ_FINAL final
Summary: Stop using MOZ_OVERRIDE and MOZ_FINAL instead use the override and final keywords. → Stop using MOZ_OVERRIDE and MOZ_FINAL in comm-central. Instead use the override and final keywords.
Assignee: nobody → archaeopteryx
Status: NEW → ASSIGNED
Attached patch patch, v3 (obsolete) — Splinter Review
Attachment #8581263 - Flags: review?(rkent)
Attachment #8581263 - Flags: review?(neil)
Is bug 810495 still needed or are all "final" keywords already placed where needed? I think I haven't seen those compiler warnings in a while.
I'm not sure, but that should probably be discussed on bug 810495. This bug is orthogonal to bug 810495.
Comment on attachment 8581263 [details] [diff] [review]
patch, v3

There is no point in waiting for reviews here.  I'll fix this.
Attachment #8581263 - Attachment is obsolete: true
Attachment #8581263 - Flags: review?(rkent)
Attachment #8581263 - Flags: review?(neil)
https://hg.mozilla.org/comm-central/rev/94adb5945ec3
Assignee: archaeopteryx → ehsan
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Since bug 1145631 was uplifted to aurora, I assume we need to do the same thing here (or at least my local builds are now breaking)
Pushed to comm-aurora https://hg.mozilla.org/releases/comm-aurora/rev/8e04fe92bd1f a=rkent
Target Milestone: --- → Thunderbird 39.0
(In reply to Kent James (:rkent) from comment #7)
> Pushed to comm-aurora
> https://hg.mozilla.org/releases/comm-aurora/rev/8e04fe92bd1f a=rkent

Oh, thanks Kent for doing this, and sorry that I forgot to do this (twice)!
You need to log in before you can comment on or make changes to this bug.