[Webkit-unassigned] [Bug 37130] Add the const modififier to HTMLLabelElement::correspondingControl
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Apr 6 10:58:20 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=37130
James Hawkins <jhawkins at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |RESOLVED
Resolution| |WONTFIX
--- Comment #7 from James Hawkins <jhawkins at chromium.org> 2010-04-06 10:58:20 PST ---
(In reply to comment #6)
> (In reply to comment #3)
> > As a matter of style, it's best to use const objects when possible.
>
> I believe this greatly overstates the matter.
>
> I think const is a quite-useful concept to make code clearer and prevent
> certain classes of programming mistakes, but in this case I don't agree.
>
> Calling innerText has many side effects, so it's not necessarily a classic
> example of a "const" operation. It can trigger a lot of work, including network
> loading.
Ok, thanks for the insights Darin. I'm going to re-close this bug.
--
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