Code review - Issue 6450099: Even when the pts are restricted to 32K values, we can still overflow computinghttps://codereview.appspot.com/2012-08-06T19:01:16+00:00rietveld
Message from unknown
2012-08-06T18:32:22+00:00reed1urn:md5:0abd6a4b3366e1576303dd38dcbc9d24
Message from unknown
2012-08-06T18:33:21+00:00reed1urn:md5:f5e25c9395765215791eff6002679ba5
Message from reed@google.com
2012-08-06T18:33:44+00:00reed1urn:md5:6d3d1812e4da8dee92c596f43040b2e3
Message from bsalomon@google.com
2012-08-06T18:46:17+00:00bsalomonurn:md5:caff7f5a58ec9478abd106a6b8fb19cc
unit test that failed on the old code?
https://codereview.appspot.com/6450099/diff/2001/src/core/SkEdge.cpp
File src/core/SkEdge.cpp (right):
https://codereview.appspot.com/6450099/diff/2001/src/core/SkEdge.cpp#newcode249
src/core/SkEdge.cpp:249: */
Nice comment.
Message from unknown
2012-08-06T18:55:32+00:00reed1urn:md5:5753ed6d78e6e1e920ded8c7d6681ad5
Message from reed@google.com
2012-08-06T18:55:49+00:00reed1urn:md5:ec70f3ccd7a58136f597b1b5c528ffc2
unittest added
Message from bsalomon@google.com
2012-08-06T19:01:16+00:00bsalomonurn:md5:24768a8e45e9d41c265b16c61f60169e
On 2012/08/06 18:55:49, reed1 wrote:
> unittest added
LGTM