[Webkit-unassigned] [Bug 76378] [EFL] Modify the code to webkit-style and adapt to C++ naming.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 19 03:35:53 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=76378


Ryosuke Niwa <rniwa at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #123090|1                           |0
        is obsolete|                            |




--- Comment #13 from Ryosuke Niwa <rniwa at webkit.org>  2012-01-19 03:35:52 PST ---
(From update of attachment 123090)
View in context: https://bugs.webkit.org/attachment.cgi?id=123090&action=review

Seems reasonable. r=me provided the change log entry is updated.

> Source/WebKit/efl/ChangeLog:3
> +        [EFL] Adapt C++ style and Modify to webkit-style.

Typo: s/Adapt/Adopt/

> Source/WebKit/efl/ChangeLog:9
> +        Modify the code to webkit-style and adapt to C++ style naming.
> +        The logic of the code is not changed.

You should mention that this patch is about renaming variables to use camelCase and spelling out variable names instead of using abbreviations. In fact, you might want to rename the bug summary accordingly.

> Source/WebKit/efl/ewk/ewk_view.cpp:220
> +#define EWK_VIEW_TYPE_CHECK(ewkView, ...)                                   \
> +    do {                                                                    \

We don't normally align \ like this; we place \ after ) followed by exactly one space.

> Source/WebKit/efl/ewk/ewk_view.cpp:2961
> + * @param window_features Features of the new window being created. If it's @c
> + * 0, it will be created a window with default features.

I don't understand this comment. Is it saying that if @c is 0, it will create a window with default features?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list