[Webkit-unassigned] [Bug 37130] Add the const modififier to HTMLLabelElement::correspondingControl

[bugzilla-daemon at webkit.org]

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


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
         Resolution|WONTFIX                     |
                 CC|                            |darin at apple.com




--- Comment #6 from Darin Adler <darin at apple.com>  2010-04-06 10:45:53 PST ---
(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.

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