Bug 52088 - Split machine stack marking functions into their own class (MachineStackMarker)
Summary: Split machine stack marking functions into their own class (MachineStackMarker)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Geoffrey Garen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-07 15:40 PST by Geoffrey Garen
Modified: 2011-01-08 19:05 PST (History)
2 users (show)

See Also:


Attachments
Patch (109.57 KB, patch)
2011-01-07 15:49 PST, Geoffrey Garen
barraclough: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Geoffrey Garen 2011-01-07 15:40:19 PST
Split machine stack marking functions into their own class (MachineStackMarker)
Comment 1 Geoffrey Garen 2011-01-07 15:49:23 PST
Created attachment 78285 [details]
Patch
Comment 2 WebKit Review Bot 2011-01-07 15:51:48 PST
Attachment 78285 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/APIShims.h', u'Source/JavaScriptCore/Android.mk', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/JavaScriptCore.gypi', u'Source/JavaScriptCore/JavaScriptCore.pro', u'Source/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/runtime/Collector.cpp', u'Source/JavaScriptCore/runtime/Collector.h', u'Source/JavaScriptCore/runtime/JSGlobalData.h', u'Source/JavaScriptCore/runtime/MachineStackMarker.cpp', u'Source/JavaScriptCore/runtime/MachineStackMarker.h']" exit_code: 1
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:41:  Alphabetical sorting problem.  [build/include_order] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:49:  "stdlib.h" already included at Source/JavaScriptCore/runtime/MachineStackMarker.cpp:28  [build/include] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:68:  Alphabetical sorting problem.  [build/include_order] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:69:  Alphabetical sorting problem.  [build/include_order] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:190:  Use 0 instead of NULL.  [readability/null] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:249:  usword_t is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:278:  Missing spaces around /  [whitespace/operators] [3]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:278:  user_count is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:281:  user_count is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:284:  user_count is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:287:  user_count is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.cpp:290:  user_count is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/JavaScriptCore/runtime/MachineStackMarker.h:34:  Code inside a namespace should not be indented.  [whitespace/indent] [4]
Total errors found: 13 in 14 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Gavin Barraclough 2011-01-07 15:58:04 PST
Comment on attachment 78285 [details]
Patch

please check the style errors, though if they're all in existing code you should need to fix before moving.
Comment 4 Geoffrey Garen 2011-01-07 17:41:50 PST
Comment on attachment 78285 [details]
Patch

Asking the commit queue to land this, since I'm out of time today.
Comment 5 WebKit Commit Bot 2011-01-08 17:28:59 PST
Comment on attachment 78285 [details]
Patch

Rejecting attachment 78285 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-sf-cq', 'apply-attachment', '--no-update', '--non-interactive', 78285]" exit_code: 2
Last 500 characters of output:
e Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj
patching file Source/JavaScriptCore/runtime/Collector.cpp
patching file Source/JavaScriptCore/runtime/Collector.h
patching file Source/JavaScriptCore/runtime/JSGlobalData.h
patching file Source/JavaScriptCore/runtime/MachineStackMarker.cpp
patching file Source/JavaScriptCore/runtime/MachineStackMarker.h

Failed to run "[u'/Projects/CommitQueue/Tools/Scripts/svn-apply', u'--reviewer', u'Gavin Barraclough', u'--force']" exit_code: 1

Full output: http://queues.webkit.org/results/7367064
Comment 6 Geoffrey Garen 2011-01-08 19:05:02 PST
Committed r75342: <http://trac.webkit.org/changeset/75342>