Bug 35091

Summary:

style-queue should not complain about identifier names with underscores under WebKit/efl/ewk/

Product:

WebKit

Reporter:

Leandro Pereira <leandro>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, gyuyoung, hamaji, rakuco

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Proposed patch	hamaji: review-
Proposed patch, with requested test case	none

Description Leandro Pereira 2010-02-18 05:13:47 PST

Our APIs use EFL style naming, so we need to use lower-cased, _-separated
names there. An example of this happenning:

https://webkit-commit-queue.appspot.com/results/283368

Comment 1 Leandro Pereira 2010-02-18 05:14:56 PST

Created attachment 49000 [details]
Proposed patch

Comment 2 Shinichiro Hamaji 2010-02-18 06:31:43 PST

Comment on attachment 49000 [details]
Proposed patch

Looks good, but could you write a test case for this? The comment above _PATH_RULES_SPECIFIER points where you should modify.

# Each string appearing in this nested list should have at least
# one associated unit test assertion.  These assertions are located,
# for example, in the test_path_rules_specifier() unit test method of
# checker_unittest.py.
_PATH_RULES_SPECIFIER = [

r- for this reason.

Comment 3 Leandro Pereira 2010-02-18 07:30:52 PST

Created attachment 49012 [details]
Proposed patch, with requested test case

Requested changes.

Note that this test will fail until WebKit/efl exists.

Comment 4 Shinichiro Hamaji 2010-02-18 18:38:35 PST

Thanks for adding the test case!

> Note that this test will fail until WebKit/efl exists.

Hmm... I thought this test doesn't check if the path exists. Also, the test seems to be passing. Will the test really fail? Otherwise, I'd like to land this patch as is.

Thanks,

Comment 5 Shinichiro Hamaji 2010-02-18 23:32:02 PST

Comment on attachment 49012 [details]
Proposed patch, with requested test case

Looks good, but setting cq- until it turns out this won't break the test.

Comment 6 Leandro Pereira 2010-02-19 12:42:11 PST

(In reply to comment #4)
> Thanks for adding the test case!
> 
> > Note that this test will fail until WebKit/efl exists.
> 
> Hmm... I thought this test doesn't check if the path exists. Also, the test
> seems to be passing. Will the test really fail? Otherwise, I'd like to land
> this patch as is.
> 

Tested it here, it seems to pass. Sorry for the false alarm.

Comment 7 Shinichiro Hamaji 2010-02-19 17:49:06 PST

Comment on attachment 49012 [details]
Proposed patch, with requested test case

Thanks for checking!

Comment 8 WebKit Commit Bot 2010-02-19 18:08:44 PST

Comment on attachment 49012 [details]
Proposed patch, with requested test case

Clearing flags on attachment: 49012

Committed r55042: <http://trac.webkit.org/changeset/55042>

Comment 9 WebKit Commit Bot 2010-02-19 18:08:49 PST

All reviewed patches have been landed.  Closing bug.