[Webkit-unassigned] [Bug 90954] [EFL] Add const to the parameter of getters in ewk_security_origin

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 12 19:04:37 PDT 2012


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





--- Comment #23 from Kihong Kwon <kihong.kwon at samsung.com>  2012-07-12 19:04:36 PST ---
(In reply to comment #21)
> IMO the patch is fine if you can explain me why remove the lazy initialization that can be a performance improvement in favor of using const.
> 
> When I wrote the API, I say that it is mostly used for manipulating database (i.e. ewk_security_origin_web_database_get_all) These getters are seldom used.
> 
> Finally, if using this const is really necessary, why not a const_cast/mutable/etc?

Security origin would be used for not only database but also other features.

Actually I'm using a Ewk_Security_Origin for the Web Notification.
In that case getters will be called frequently, that's why I moved initialization to *_new.

-- 
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