[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 #3 from Ryuan Choi <ryuan.choi at samsung.com>  2011-02-23 15:54:35 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]
> 
> > Source/WebKit/efl/ewk/ewk_contextmenu.cpp:57
> > +    Ewk_Context_Menu* submenu; /** contains the pointer to the submenu of the item */
> 
> Please add "<" to "/**"
> 
> > Source/WebKit/efl/ewk/ewk_contextmenu.h:79
> > +    EWK_CONTEXT_MENU_ITEM_TAG_SPELLING_MENU, /**< spelling or spelling/grammar sub-menu */
> 
> This is not sync with ewk_cookie.cpp. Which one is better?  "/**<" or "///"
> 
> > 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 "///"

IMO,  "/**<" is better because EFL is based on C application.

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