[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