tree d209b88ee5909d928de666b31b6aa2bd48469c0d
parent 0f9df470ca1da36bcc515b7ef3b11012690d2845
author Mike Klein <mtklein@google.com> 1575318982 -0600
committer Skia Commit-Bot <skia-commit-bot@chromium.org> 1575323839 +0000

tweak for GCC, memcpy() -> for-loop

I think GCC with optimization is getting confused about what field of
this union is active, and then triggers a crazy warning about
memcpy()'ing (size_t)-1 bytes.

Reading the rules about unions, I think it might be that really only "="
is ever able to switch which union field is active, and our memcpy()
here really doesn't technically work to change the active field.

Change-Id: Ibe33143161935d5e39b9a2f5d35560e4961056e0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/257444
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>