RESOLVED FIXED 35968
[Qt] QScriptValuePrivate::States has naming issues 
https://bugs.webkit.org/show_bug.cgi?id=35968
Summary [Qt] QScriptValuePrivate::States has naming issues
Jędrzej Nowacki
Reported 2010-03-10 05:37:53 PST
- The enum States has invalid name, according to Qt coding style it should be "State" as enum's elements are not flags that could be or'ed or and'ed. - JSNative should be renamed to JSPrimitive as "primitive value" is a corresponding name in ECMA Script standard ("A primitive value is a member of one of the following built-in types: Undefined, Null, Boolean, Number, and String").
Attachments
Fix v1 (9.08 KB, patch)
2010-03-10 06:08 PST, Jędrzej Nowacki 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jędrzej Nowacki
Comment 1 2010-03-10 06:08:36 PST
Created attachment 50397 [details] Fix v1
WebKit Commit Bot
Comment 2 2010-03-11 21:41:48 PST
Comment on attachment 50397 [details] Fix v1 Clearing flags on attachment: 50397 Committed r55887: <http://trac.webkit.org/changeset/55887>
WebKit Commit Bot
Comment 3 2010-03-11 21:41:52 PST
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.