tree 83d8d3dc364db92f5e3b62b758e4d714c582cd7d
parent c0be9bb8bc1754c9de91b9b22f8f578fa9b39207
author John Stiles <johnstiles@google.com> 1700591513 -0500
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> 1700592048 +0000

Improve R+G+B op fusing logic.

Previously, we checked for R+G+B ops on every iteration through
the `select_curve_ops` loop because we needed to substitute the
fused RGB op before the A op was added. Now, we process the
channels in reverse order--yielding A/B/G/R instead of R/G/B/A.
With RGB at the end of the list, it is easy to substitute out
a fused op (simply move the cursor back and change the op) at
the end of the loop instead of checking on each loop iteration.

This isn't a big deal but the RGB check inside of the curve-
picking inner loop never felt like a good solution to me.

Change-Id: I978346ceb1b12e116a64af06e909e5adbfd7eedc
Reviewed-on: https://skia-review.googlesource.com/c/skcms/+/780816
Commit-Queue: John Stiles <johnstiles@google.com>
Auto-Submit: John Stiles <johnstiles@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>