Created attachment 421664 [details] [diff] [review] patch, v1 We have the ability to do "nsIFoo.*, -nsIFoo.bar" now, but that doesn't currently work if bar isn't valid for quickstubs. This patch changes things around a bit so that the quickstub list is built up first, taking into account all the additions and removals, and then check the full list at once for validity. This means that methods that can't be quickstubbed still must be explicitly listed as -, but it makes quickstubbing interfaces that have all good methods and one or two bad ones much simpler. One quirk is that we add the 'traceable' attr on a member, whereas coming from the parser it could only have 'readonly' or 'optional_argc', so I had to add that to the list of recognized attrnames.
Attachment #421664 - Flags: review?(jorendorff)
Attachment #421664 - Flags: review?(jorendorff) → review+
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.