[webkit-reviews] review requested: [Bug 54693] [EFL] Fix coding style errors in RenderThemeEfl.h : [Attachment 82874] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Feb 17 16:19:20 PST 2011
Gyuyoung Kim <gyuyoung.kim at samsung.com> has asked for review:
Bug 54693: [EFL] Fix coding style errors in RenderThemeEfl.h
https://bugs.webkit.org/show_bug.cgi?id=54693
Attachment 82874: Patch
https://bugs.webkit.org/attachment.cgi?id=82874&action=review
------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
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.
More information about the webkit-reviews
mailing list