WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
143384
Some JSC Options refactoring and enhancements
https://bugs.webkit.org/show_bug.cgi?id=143384
Summary
Some JSC Options refactoring and enhancements
Mark Lam
Reported
2015-04-03 13:09:22 PDT
Create a better encapsulated Option class to make working with options easier. This is a building block towards a JIT policy scaling debugging option I will introduce later. This work entails: 1. Convert Options::Option into a public class Option (who works closely with Options). 2. Convert Options::EntryType into an enum class Options::Type and make it public. 3. Renamed Options::OPT_<option name> to Options::<option name>ID because it reads better. 4. Add misc methods to class Option to make it more useable.
Attachments
the patch.
(10.85 KB, patch)
2015-04-03 13:35 PDT
,
Mark Lam
benjamin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Mark Lam
Comment 1
2015-04-03 13:35:55 PDT
Created
attachment 250095
[details]
the patch.
WebKit Commit Bot
Comment 2
2015-04-03 13:37:17 PDT
Attachment 250095
[details]
did not pass style-queue: ERROR: Source/JavaScriptCore/runtime/Options.h:324: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:331: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:332: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:333: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:334: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:335: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:336: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] ERROR: Source/JavaScriptCore/runtime/Options.h:337: enum members should use InterCaps with an initial capital letter. [readability/enum_casing] [4] Total errors found: 8 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Mark Lam
Comment 3
2015-04-03 13:39:48 PDT
(In reply to
comment #2
)
>
Attachment 250095
[details]
did not pass style-queue:
FYI, these style violations are intentional and needed to be able to automtically synthesize Options functions via macros. These warning only surfaced because I moved the Options::EntryType enums into the new Options::Type enums.
Benjamin Poulain
Comment 4
2015-04-03 13:59:10 PDT
Comment on
attachment 250095
[details]
the patch. Sure, rs=me
Mark Lam
Comment 5
2015-04-03 14:03:06 PDT
Thanks. Landed in
r182331
: <
http://trac.webkit.org/r182331
>.
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