[Webkit-unassigned] [Bug 129486] Implement Math.hypot

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 3 02:18:54 PST 2014


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





--- Comment #4 from Tibor Mészáros <tmeszaros.u-szeged at partner.samsung.com>  2014-03-03 02:15:58 PST ---
(In reply to comment #3)
> (From update of attachment 225476 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=225476&action=review
> 
> >> Source/JavaScriptCore/runtime/MathObject.cpp:189
> >> +    for (unsigned k = 0; k < argsCount; ++k) {
> > 
> > Why "k" when we use "-" everywhere else?
> 
> I meant: Why "k" when we use "i" elsewhere in WebKit code for this?

Hi!

Thanks for the detailed review, I'm going to create a new patch today, to cover your remarks.

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