[Webkit-unassigned] [Bug 79362] [EFL] Remove a duplicate allocation of matrix entry.

[bugzilla-daemon at webkit.org]

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





--- Comment #10 from Grzegorz Czajkowski <g.czajkowski at samsung.com>  2012-03-12 08:03:46 PST ---
(In reply to comment #9)
> (In reply to comment #7)
> > This API is obvious for purpose. Does it need more description?
> If you want to add function description, in my humble opinion, it is better to adhere doxyzen rule regardless of internal | public.  ewk_view.cpp also added function description as below,
> /**
>  * @internal                                                              
>  *
>  * @param ewkView View.                                                   
>  * @param message String to show on console.
>  * @param lineNumber Line number.                                         
>  * @sourceID Source id.
>  *
>  */ 
> void ewk_view_add_console_message(Evas_Object* ewkView, const char* message, unsigned int lineNumber, const char* sourceID)
> Grzegorz, how do you think about it ?

If this function requires documentation I also will use doxygen. Generally backing store doesn't have extended documentation (its purpose is rather for internal use).In my opinion there is no needed to add doxygen documentation for each backing store function.

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