Bug 54693 – [EFL] Fix coding style errors in RenderThemeEfl.h

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit EFL

[Gyuyoung Kim]

When I run check-webkit-style script with RenderThemeEfl.h, there are many style erros like ewk_view.h. 

Source/WebCore/platform/efl/RenderThemeEfl.h:37:  Ecore_Evas is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/WebCore/platform/efl/RenderThemeEfl.h:38:  Evas_Object is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/WebCore/platform/efl/RenderThemeEfl.h:104:  The parameter name "style" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:104:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:149:  The parameter name "size" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:181:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:182:  The parameter name "o" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:182:  The parameter name "states" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:183:  The parameter name "o" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:183:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:183:  The parameter name "i" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:183:  The parameter name "rect" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:208:  The parameter name "desc" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:209:  The parameter name "o" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:209:  The parameter name "desc" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:226:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:226:  The parameter name "size" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:231:  The parameter name "ce" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:231:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:232:  The parameter name "ce" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:233:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:233:  The parameter name "size" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:234:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:234:  The parameter name "ce" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:235:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:235:  The parameter name "size" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/WebCore/platform/efl/RenderThemeEfl.h:235:  The parameter name "ce" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 27 in 1 files

[Gyuyoung Kim]

Created attachment 82874 [details]
Patch

I make a patch for this. If parameter type represents parameter's meaning well, I think we don't need to add parameter unnecessarily.
Other port add just parameter type to function definition. (See also RenderThemeGtk.h, RenderThemeQt.h)

However, there are still style errors regarding underscore.

Source/WebCore/platform/efl/RenderThemeEfl.h:37:  Ecore_Evas is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
Source/WebCore/platform/efl/RenderThemeEfl.h:38:  Evas_Object is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]

I don't know how to fix this errors yet.

[Gyuyoung Kim]

Comment on attachment 82874 [details]
Patch

There was problem in eflews

[Gyuyoung Kim]

Created attachment 82880 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 82880 [details]
Patch

Clearing flags on attachment: 82880

Committed r79080: <http://trac.webkit.org/changeset/79080>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.