Remove inout params from saturation helper func in XferEffect.

Remove inout params from saturation helper func in XferEffect.

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

[bsalomon, 2013-04-29 16:58:35]

This seems to fix the PVR issues in the saturation xfermode.

[robertphillips, 2013-04-29 17:04:02]

lgtm + 1 question

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp
File src/core/SkXfermode.cpp (right):

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp#ne...
src/core/SkXfermode.cpp:1310: "\t}\n",
Why not just put this in the string?

[jvanverth1, 2013-04-29 17:04:24]

lgtm

[bsalomon, 2013-04-29 17:07:02]

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp
File src/core/SkXfermode.cpp (right):

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp#ne...
src/core/SkXfermode.cpp:1310: "\t}\n",
On 2013/04/29 17:04:02, robertphillips wrote:
> Why not just put this in the string?

Laziness I suppose. I can never remember whether the .0s are required here.
(just checked now and they're not).

[bsalomon, 2013-04-29 17:12:22]

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp
File src/core/SkXfermode.cpp (right):

https://codereview.chromium.org/14535004/diff/3001/src/core/SkXfermode.cpp#ne...
src/core/SkXfermode.cpp:1310: "\t}\n",
On 2013/04/29 17:04:02, robertphillips wrote:
> Why not just put this in the string?

Done.

[commit-bot: I haz the power, 2013-04-29 17:12:30]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/bsalomon@google.com/14535004/10001

[commit-bot: I haz the power, 2013-04-29 17:36:17]

Change committed as 8909