[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


Gyuyoung Kim <gyuyoung.kim at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |g.czajkowski at samsung.com,
                   |                            |gyuyoung.kim at samsung.com




--- Comment #9 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2012-03-12 06:05:17 PST ---
(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 ?

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