That's confusing. Make it clear we're working with 3 points == 9 scalars.

That's confusing.  Make it clear we're working with 3 points == 9 scalars.

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=13216

[reed1, 2014-01-28 13:20:35]

https://codereview.chromium.org/148383009/diff/1/src/core/SkGeometry.cpp
File src/core/SkGeometry.cpp (right):

https://codereview.chromium.org/148383009/diff/1/src/core/SkGeometry.cpp#newc...
src/core/SkGeometry.cpp:1341: static void p3d_interp(const SkScalar src[9],
SkScalar dst[9], SkScalar t) {
Maybe they should be [7] ?

[mtklein_C, 2014-01-28 13:37:31]

On 2014/01/28 13:20:35, reed1 wrote:
> https://codereview.chromium.org/148383009/diff/1/src/core/SkGeometry.cpp
> File src/core/SkGeometry.cpp (right):
> 
>
https://codereview.chromium.org/148383009/diff/1/src/core/SkGeometry.cpp#newc...
> src/core/SkGeometry.cpp:1341: static void p3d_interp(const SkScalar src[9],
> SkScalar dst[9], SkScalar t) {
> Maybe they should be [7] ?

Done.  I added a little note that would have left me less confused (but no less
terrified).

[reed1, 2014-01-28 14:07:51]

lgtm

[commit-bot: I haz the power, 2014-01-28 14:23:49]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/148383009/20001

[commit-bot: I haz the power, 2014-01-28 14:36:55]

Change committed as 13216