Closed Bug 1156790 Opened 7 years ago Closed 7 years ago

mark various nsCOMPtr_helper-esque classes as final

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

No description provided.
I thought this might help devirtualize some of the operator() calls, but we
might already be devirtualizing them.  In which case this patch is really
encouraging people to not do weird things with these classes.
Attachment #8595369 - Flags: review?(continuation)
Comment on attachment 8595369 [details] [diff] [review]
mark various nsCOMPtr_helper-esque classes as final

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

huzzah
Attachment #8595369 - Flags: review?(continuation) → review+
https://hg.mozilla.org/mozilla-central/rev/dcc5991232c8
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.