Bug 143384 – Some JSC Options refactoring and enhancements

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Mark Lam]

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.

[Mark Lam]

Created attachment 250095 [details]
the patch.

[WebKit Commit Bot]

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]

(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 on attachment 250095 [details]
the patch.

Sure, rs=me

[Mark Lam]

Thanks.  Landed in r182331: <http://trac.webkit.org/r182331>.