RESOLVED FIXED65731
Add ParentClass typedef in all JSC classes 
https://bugs.webkit.org/show_bug.cgi?id=65731
Summary Add ParentClass typedef in all JSC classes
Mark Hahnenberg
Reported 2011-08-04 16:05:32 PDT
Eventually we'd like to get rid of the vtable pointer in all of the JSC classes and use explicit super class calls for all inherited methods. To make that future task simpler (and more template-able), it would be a good idea to add a typedef to each class by which we can uniformly refer to its parent class in the inheritance hierarhcy.
Attachments
Patch (73.86 KB, patch)
2011-08-05 13:41 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (74.35 KB, patch)
2011-08-05 14:33 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (98.05 KB, patch)
2011-08-08 16:41 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2011-08-05 13:41:34 PDT
Created attachment 103100 [details] Patch
Mark Hahnenberg
Comment 2 2011-08-05 14:33:00 PDT
Created attachment 103115 [details] Patch
Gavin Barraclough
Comment 3 2011-08-07 13:55:17 PDT
Hi Mark, Did you want this patch reviewing, or is this still work in progress? - the review flag isn't set & it's unclear from the description. G.
Mark Hahnenberg
Comment 4 2011-08-08 08:21:22 PDT
(In reply to comment #3) > Hi Mark, > > Did you want this patch reviewing, or is this still work in progress? - the review flag isn't set & it's unclear from the description. > > G. Still trying to get Windows to build. Getting some mysterious template-related errors on just Windows :-(
Mark Hahnenberg
Comment 5 2011-08-08 16:41:44 PDT
Created attachment 103312 [details] Patch
Oliver Hunt
Comment 6 2011-08-09 12:44:58 PDT
Comment on attachment 103312 [details] Patch r=me
WebKit Review Bot
Comment 7 2011-08-09 13:46:34 PDT
Comment on attachment 103312 [details] Patch Clearing flags on attachment: 103312 Committed r92706: <http://trac.webkit.org/changeset/92706>
WebKit Review Bot
Comment 8 2011-08-09 13:46:39 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.