[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 03:49:52 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=90954
--- Comment #18 from Gyuyoung Kim <gyuyoung.kim at samsung.com> 2012-07-12 03:49:51 PST ---
(In reply to comment #17)
> (In reply to comment #16)
> > Created an attachment (id=151877)
--> (https://bugs.webkit.org/attachment.cgi?id=151877&action=review) [details] [details]
> > Patch
>
> Thanks for changes. Looks fine for me.
> I'd add notes to ChangeLog that you are moving initialization of strings for protocol and host to the *_new method. That change allows to add const modifier for getters methods. Additionally null checks are added to the getters.
After updating ChangeLog according to Grezegorz's suggestion, I will give LGTM.
--
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