WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 37367
Make Generated V8 code adhere to webkit style
https://bugs.webkit.org/show_bug.cgi?id=37367
Summary
Make Generated V8 code adhere to webkit style
Yaar Schnitman
Reported
2010-04-09 15:29:56 PDT
I almost adheres, but there are a few things it doesn't.
Attachments
patch
(50.57 KB, patch)
2010-04-20 15:58 PDT
,
Nate Chapin
dglazkov
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Nate Chapin
Comment 1
2010-04-20 15:58:32 PDT
Created
attachment 53901
[details]
patch This also fixes a bunch of style violations the TestObj.idl doesn't trigger. I ran check-webkit-style over the entire generated bindings, and there now is only a single violation (in V8DOMWindow.cpp), which would be non-trivial to resolve and probably cause the script to become substantially less readable.
Dimitri Glazkov (Google)
Comment 2
2010-04-20 16:05:54 PDT
Comment on
attachment 53901
[details]
patch awesome. Thanks!
Nate Chapin
Comment 3
2010-04-20 16:29:07 PDT
http://trac.webkit.org/changeset/57936
WebKit Review Bot
Comment 4
2010-04-20 20:53:39 PDT
http://trac.webkit.org/changeset/57936
might have broken Qt Windows 32-bit Debug The following changes are on the blame list:
http://trac.webkit.org/changeset/57936
http://trac.webkit.org/changeset/57937
http://trac.webkit.org/changeset/57938
http://trac.webkit.org/changeset/57939
http://trac.webkit.org/changeset/57940
http://trac.webkit.org/changeset/57941
http://trac.webkit.org/changeset/57942
http://trac.webkit.org/changeset/57943
http://trac.webkit.org/changeset/57944
http://trac.webkit.org/changeset/57945
http://trac.webkit.org/changeset/57946
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug