The dash counts were signed, but were used in an unsigned way such as indexing into an array.

We are unlikely to need that extra bit. However, it also calls SkAlign2(). SkAlign2() is implemented with bit shifts, which cause undefined behavior on signed ints.


In addition, we want to replace SkAlign?() macros with proper functions. The current set doesn't work on pointer types. Moving to the proper functions requires the calling points be fixed.