Bug 49519

Summary: check-webkit-style unit tests has some duplicate boilerplate code.
Product: WebKit Reporter: David Levin <levin>
Component: Tools / TestsAssignee: David Levin <levin>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dbates, eric, hamaji, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 49513    
Bug Blocks:    
Attachments:
Description Flags
Patch hamaji: review+, levin: commit-queue-

Description David Levin 2010-11-14 16:08:42 PST
Duplicate code is bad.
Comment 1 David Levin 2010-11-14 16:09:37 PST
See https://bugs.webkit.org/show_bug.cgi?id=49513#c2
Comment 2 David Levin 2010-12-16 08:19:57 PST
Created attachment 76771 [details]
Patch
Comment 3 Shinichiro Hamaji 2010-12-16 09:28:48 PST
Comment on attachment 76771 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=76771&action=review

Looks good.

> WebKitTools/Scripts/webkitpy/style/checkers/cpp.py:2869
> +

Do we need this blank line?
Comment 4 David Levin 2010-12-16 09:29:46 PST
Comment on attachment 76771 [details]
Patch

Will address feedback and land.
Comment 5 David Levin 2010-12-16 10:45:54 PST
Committed r74200: <http://trac.webkit.org/changeset/74200>
Comment 6 WebKit Review Bot 2010-12-16 15:39:38 PST
http://trac.webkit.org/changeset/74200 might have broken Leopard Intel Debug (Tests)