Bug 134884

Summary:

check-webkit-style should apply C++ rules to Objective-C++ files

Product:

WebKit

Reporter:

Daniel Bates <dbates>

Component:

Tools / Tests

Assignee:

Daniel Bates <dbates>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, darin, ddkilzer, glenn, joepeck

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch and unit tests	joepeck: review+

Description Daniel Bates 2014-07-14 09:08:56 PDT

Currently check-webkit-style considers Objective-C++ files as Objective-C files and hence doesn't apply rules that are applicable to C++ code. We should teach check-webkit-style to apply its C++ rules to Objective-C++ files (if applicable - see remark (*)) to identify Objective-C++ code that doesn't conform to the WebKit Code Style Guidelines.

(*) At the time of writing, we allow Objective-C code in an Objective-C++ file to use NULL for a null pointer instead of the C++ idiom nullptr per <http://www.webkit.org/coding/coding-style.html#zero-null>.

Comment 1 Daniel Bates 2014-07-14 09:21:05 PDT

Created attachment 234856 [details]
Patch and unit tests

Comment 2 Joseph Pecoraro 2014-07-14 11:01:33 PDT

Comment on attachment 234856 [details]
Patch and unit tests

r=me, nice!

Comment 3 Daniel Bates 2014-07-14 15:23:11 PDT

Committed r171088: <http://trac.webkit.org/changeset/171088>