Bug 119359 - More cleanup in PropertySlot
Summary: More cleanup in PropertySlot
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Gavin Barraclough
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-31 16:55 PDT by Gavin Barraclough
Modified: 2013-07-31 18:08 PDT (History)
1 user (show)

See Also:


Attachments
fix (14.54 KB, patch)
2013-07-31 16:56 PDT, Gavin Barraclough
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gavin Barraclough 2013-07-31 16:55:29 PDT
m_slotBase is overloaded to store the (receiver) thisValue and the object that contains the property,
This is confusing, and means that slotBase cannot be typed correctly (can only be a JSObject).
Comment 1 Gavin Barraclough 2013-07-31 16:56:58 PDT
Created attachment 207888 [details]
fix
Comment 2 Geoffrey Garen 2013-07-31 16:59:19 PDT
Comment on attachment 207888 [details]
fix

r=me
Comment 3 WebKit Commit Bot 2013-07-31 16:59:44 PDT
Attachment 207888 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/dfg/DFGRepatch.cpp', u'Source/JavaScriptCore/jit/JITStubs.cpp', u'Source/JavaScriptCore/runtime/JSObject.cpp', u'Source/JavaScriptCore/runtime/JSObject.h', u'Source/JavaScriptCore/runtime/PropertySlot.cpp', u'Source/JavaScriptCore/runtime/PropertySlot.h', u'Source/JavaScriptCore/runtime/SparseArrayValueMap.cpp', u'Source/JavaScriptCore/runtime/SparseArrayValueMap.h']" exit_code: 1
Source/JavaScriptCore/runtime/PropertySlot.h:62:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/PropertySlot.h:62:  The parameter name "propertyName" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/PropertySlot.h:63:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 3 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Gavin Barraclough 2013-07-31 18:08:17 PDT
Committed revision 153556.