[webkit-reviews] review granted: [Bug 87915] [chromium] Fix min/max bounds error in CCMathUtil.cpp : [Attachment 144967] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 30 18:43:04 PDT 2012


James Robinson <jamesr at chromium.org> has granted Shawn Singh
<shawnsingh at chromium.org>'s request for review:
Bug 87915: [chromium] Fix min/max bounds error in CCMathUtil.cpp
https://bugs.webkit.org/show_bug.cgi?id=87915

Attachment 144967: Patch
https://bugs.webkit.org/attachment.cgi?id=144967&action=review

------- Additional Comments from James Robinson <jamesr at chromium.org>
numeric_limits<float>::min() is actually something like the least positive
representable number (can't remember if it's a denormal or not).  Pretty much
never what you want.

Looks good, do we expect any impact on pages?


More information about the webkit-reviews mailing list