Closed Bug 896275 Opened 7 years ago Closed 7 years ago

Use MOZ_THIS_IN_INITIALIZER_LIST in dom/

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
Sorry about the includes; I wrote this before you suggested avoiding them.
Attachment #778975 - Flags: review?(mounir)
Attachment #778975 - Flags: review?(mounir) → review+
https://hg.mozilla.org/mozilla-central/rev/dc2f8b382241
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Depends on: 898311
Is it necessary to move MOZ_WIDGET_ includes down? See 898311.
man, why did you change all those includes ?
(In reply to Jan Varga [:janv] from comment #3)
> man, why did you change all those includes ?

He has a tool reordering the #include in some kind of alphabetical order IIRC.
(In reply to Mounir Lamouri (:mounir) from comment #4)
> (In reply to Jan Varga [:janv] from comment #3)
> > man, why did you change all those includes ?
> 
> He has a tool reordering the #include in some kind of alphabetical order
> IIRC.

So, is this the official style/scheme that we should try to follow ?
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.