Closed Bug 1034923 Opened 7 years ago Closed 7 years ago
Event Gen .py code generator should not generate dangerous public destructors in refcounted classes
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.
Hah - I was looking for "accessibility".
Hmm. It seems to be okay to just make the dtor private here, which seems odd to me.
Assignee: nobody → continuation
Attachment #8463439 - Flags: review?(trev.saunders)
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+
Thanks for the review. https://hg.mozilla.org/integration/mozilla-inbound/rev/2fd5026e35f1
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.