Last Comment Bug 773637 - Kill NS_SCRIPTABLE and friends
: Kill NS_SCRIPTABLE and friends
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Rewriting and Analysis (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla17
Assigned To: Joshua Cranmer [:jcranmer]
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-13 07:15 PDT by Joshua Cranmer [:jcranmer]
Modified: 2012-07-24 02:59 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Part 1: Kill NS_*PARAM (115.85 KB, patch)
2012-07-13 07:16 PDT, Joshua Cranmer [:jcranmer]
ehsan: review+
Details | Diff | Splinter Review
Part 2: Kill NS_SCRIPTABLE (64.53 KB, patch)
2012-07-13 07:17 PDT, Joshua Cranmer [:jcranmer]
ehsan: review+
Details | Diff | Splinter Review
Remove it from comm-central (2.17 KB, patch)
2012-07-13 07:19 PDT, Joshua Cranmer [:jcranmer]
mconley: review+
Pidgeot18: checkin+
Details | Diff | Splinter Review

Description Joshua Cranmer [:jcranmer] 2012-07-13 07:15:36 PDT
No analysis in the new static analysis framework is planning on using NS_SCRIPTABLE and NS_INPARAM/NS_OUTPARAM/NS_INOUTPARAM attributes. We should just remove them.
Comment 1 Joshua Cranmer [:jcranmer] 2012-07-13 07:16:59 PDT
Created attachment 641863 [details] [diff] [review]
Part 1: Kill NS_*PARAM
Comment 2 Joshua Cranmer [:jcranmer] 2012-07-13 07:17:35 PDT
Created attachment 641864 [details] [diff] [review]
Part 2: Kill NS_SCRIPTABLE
Comment 3 Joshua Cranmer [:jcranmer] 2012-07-13 07:19:39 PDT
Created attachment 641866 [details] [diff] [review]
Remove it from comm-central

Comm-central actually bothered to use these?
Comment 4 Joshua Cranmer [:jcranmer] 2012-07-18 11:29:47 PDT
Comment on attachment 641866 [details] [diff] [review]
Remove it from comm-central

https://hg.mozilla.org/comm-central/rev/ce86b74dc946

Note You need to log in before you can comment on or make changes to this bug.