De-COM nsIXPConnect
Categories
(Core :: XPConnect, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox107 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
It hasn't been scriptable in years, so there's no need for it to be defined in an IDL, or for any of its methods to be virtual. Among other things, de-COMming it will make it much easier to query in Searchfox.
Assignee | ||
Comment 1•2 years ago
|
||
They haven't been scriptable in years, so there's no need for them to go
through XPIDL or for the methods to be virtual.
Comment 2•2 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:kmag, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.
Comment 3•2 years ago
|
||
I spoke to kmag about this earlier this week. He hasn't had a chance to update his patch yet, but it is on his mind.
Pushed by maglione.k@gmail.com: https://hg.mozilla.org/integration/autoland/rev/b5b51c1be2f5 Decom nsIXPConnect and friends. r=mccr8
Comment 5•2 years ago
|
||
Backed out for causing build bustages on XPCWrappedJS.cpp
- Backout link
- Push with failures
- Failure Log
- Failure line: /builds/worker/checkouts/gecko/js/xpconnect/src/XPCWrappedJS.cpp(598,24): error: function declared 'stdcall' here was previously declared without calling convention
Assignee | ||
Updated•2 years ago
|
Pushed by maglione.k@gmail.com: https://hg.mozilla.org/integration/autoland/rev/124085e3eed1 Decom nsIXPConnect and friends. r=mccr8
Comment 7•2 years ago
|
||
bugherder |
Description
•