[Webkit-unassigned] [Bug 55026] [EFL] Doxygen documentation for ewk_window_features and ewk_context_menu

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Grzegorz <g.czajkowski at samsung.com>  2011-03-03 05:48:48 PST ---
(In reply to comment #2)
> (From update of attachment 83461 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=83461&action=review
> 
> > Source/WebKit/efl/ChangeLog:5
> > +        [EF] Doxygen documentation for ewk_window_features and ewk_context_menu
> 
> [EF] => [EFL]

fixed

> 
> > Source/WebKit/efl/ewk/ewk_contextmenu.cpp:57
> > +    Ewk_Context_Menu* submenu; /** contains the pointer to the submenu of the item */
> 
> Please add "<" to "/**"

added

> 
> > Source/WebKit/efl/ewk/ewk_contextmenu.h:84
> > +    EWK_CONTEXT_MENU_ITEM_TAG_FONT_MENU, /**< font sub-menu */
> 
> This is not sync with ewk_cookie.cpp. Which one is better?  "/**<" or "///"

All "/// comment" were replaced to "/**< comment */" or to "/** comment */" in ewk_context_menu/ewk_widow_features.
"/**< comment */ is dedicated for struct member
"/** comment */ is dedicated for structs/enums description

I will fix this in ewk_cookies soon because it's better style in C applications

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