RESOLVED FIXED 49446
check-webkit-style function detection doesn't detect indented functions declaractions. 
https://bugs.webkit.org/show_bug.cgi?id=49446
Summary check-webkit-style function detection doesn't detect indented functions decla...
David Levin
Reported 2010-11-12 07:58:53 PST
See summary.
Attachments
Patch (5.22 KB, patch)
2010-11-12 08:01 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch (5.22 KB, patch)
2010-11-12 08:04 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch (6.95 KB, patch)
2010-11-12 09:36 PST, David Levin 		hamaji: review+
commit-queue: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2010-11-12 08:01:11 PST
Created attachment 73744 [details] Patch
David Levin
Comment 2 2010-11-12 08:04:10 PST
Created attachment 73746 [details] Patch
David Levin
Comment 3 2010-11-12 08:07:45 PST
Comment on attachment 73746 [details] Patch Whoops, I meant cq? not cq+.
WebKit Commit Bot
Comment 4 2010-11-12 08:24:38 PST
Comment on attachment 73746 [details] Patch Rejecting patch 73746 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=abarth-cq-sl', 'build-and-test', '--no-clean', '--no-update', '--test', '--non-interactive']" exit_code: 2 Last 500 characters of output: tTools/Scripts/webkitpy/style/checkers/cpp.py", line 1211, in check_for_function_lengths ending_line_number = close_expression(clean_lines, start_line_number, open_brace_index)[1] File "/Users/abarth/git/webkit-queue/WebKitTools/Scripts/webkitpy/style/checkers/cpp.py", line 643, in close_expression line = clean_lines.elided[line_number] IndexError: list index out of range ---------------------------------------------------------------------- Ran 692 tests in 8.902s FAILED (errors=8) Full output: http://queues.webkit.org/results/5753022
David Levin
Comment 5 2010-11-12 09:11:54 PST
Comment on attachment 73746 [details] Patch Tests don't pass.
David Levin
Comment 6 2010-11-12 09:36:14 PST
Created attachment 73752 [details] Patch
Shinichiro Hamaji
Comment 7 2010-11-13 02:15:45 PST
Comment on attachment 73752 [details] Patch Looks good.
WebKit Commit Bot
Comment 8 2010-11-13 19:26:25 PST
Comment on attachment 73752 [details] Patch Rejecting patch 73752 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sl', 'apply-attachment', '--force-clean', '--non-interactive', 73752]" exit_code: 2 Last 500 characters of output: succeeded at 612 (offset 1 line). Hunk #3 succeeded at 622 (offset 1 line). Hunk #4 succeeded at 639 (offset 1 line). Hunk #5 succeeded at 1174 (offset 1 line). Hunk #6 FAILED at 1210. 1 out of 6 hunks FAILED -- saving rejects to file WebKitTools/Scripts/webkitpy/style/checkers/cpp.py.rej patching file WebKitTools/Scripts/webkitpy/style/checkers/cpp_unittest.py Failed to run "[u'/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Shinichiro Hamaji', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/6020030
David Levin
Comment 9 2010-11-13 20:11:42 PST
Committed as http://trac.webkit.org/changeset/71976
Note You need to log in before you can comment on or make changes to this bug.