Closed
Bug 1034923
Opened 10 years ago
Closed 10 years ago
AccEventGen.py code generator should not generate dangerous public destructors in refcounted classes
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: bjacob, Assigned: mccr8)
References
Details
Attachments
(1 file)
(deleted),
patch
|
tbsaunde
:
review+
|
Details | Diff | Splinter Review |
In bug 1028588 we removed dangerous public destructors of XPCOM-refcounted classes outside of a finite whitelist, see HasDangerousPublicDestructor. Now we are going over the entries in this whitelist. One such occurrence is in the AccEventGen.py code generator. It currently generates classes with public destructors, even though the classes are reference-counted. For that reason, in bug 1028588 we are making this code generator also generate HasDangerousPublicDestructor specializations for these classes. But what we really want is no dangerous public destructors at all.
Updated•10 years ago
|
Component: DOM: Events → Disability Access APIs
Assignee | ||
Comment 2•10 years ago
|
||
Hmm. It seems to be okay to just make the dtor private here, which seems odd to me.
Assignee: nobody → continuation
Assignee | ||
Comment 3•10 years ago
|
||
try: https://tbpl.mozilla.org/?tree=Try&rev=09ff07ea3d51
Attachment #8463439 -
Flags: review?(trev.saunders)
Comment 4•10 years ago
|
||
Comment on attachment 8463439 [details] [diff] [review] Make AccEventGen.py code generator generate classes with private destructors. thanks!
Attachment #8463439 -
Flags: review?(trev.saunders) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Thanks for the review. https://hg.mozilla.org/integration/mozilla-inbound/rev/2fd5026e35f1
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2fd5026e35f1
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•