DescriptionDon't do a pointless << 0.
It's very common (universal?) that alpha is the top byte.
You'd hope the compiler would remove the left shift then,
but I've seen Clang just do a dumb left shift of zero. :(
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/8d029a4ebeda9d99c40b864e9c93c55be58b05af
Patch Set 1 #
Messages
Total messages: 11 (4 generated)
|