Summary: | JavaParameter should be removed | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Steve Block <steveblock> | ||||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | andreip, commit-queue, jorlow, steveblock | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 55383, 55774 | ||||||||||
Attachments: |
|
Description
Steve Block
2011-03-04 06:36:35 PST
Created attachment 84740 [details]
Patch
LGTM with a minor nit > 64: for (int i = 0; i < numParameters; i++) Here 'i' is an int. > 123: for (unsigned int i = 0; i < m_parameters.size(); i++) Here 'i' is an unsigned. Maybe make this consistent? Created attachment 84750 [details]
Patch
Thanks, LGTM Comment on attachment 84750 [details]
Patch
I don't really understand this stuff, but I trust Andrei's LGTM...r=me
Comment on attachment 84750 [details] Patch Rejecting attachment 84750 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-8', 'build-..." exit_code: 2 Last 500 characters of output: .... inspector/audits . inspector/console ...................... inspector/debugger .................... inspector/editor ... inspector/elements ................ inspector/extensions ....... inspector/protocol . inspector/styles ............. inspector/timeline ........... java . java/argument-to-object-type.html -> crashed Exiting early after 1 failures. 12852 tests run. 435.17s total testing time 12851 test cases (99%) succeeded 1 test case (<1%) crashed 8 test cases (<1%) had stderr output Full output: http://queues.webkit.org/results/8086895 Created attachment 84936 [details]
Patch
Fixed potential crash due to CString local going out of scope when using String::utf8()::data() Comment on attachment 84936 [details]
Patch
Retaining jorlow's r+
Committed r80467: <http://trac.webkit.org/changeset/80467> |