don't memset TriangulationVertex
GCC gets nervous here, and I think correctly so. The type is
non-trivial (because the linked list has a constructor from its field
initializers) and we're bypassing that instead with a memset(). That'd
be _close_ to fine, but I think less clear than just =
TriangulationVertex{}.
What's really weird is that we're only zeroing it in Debug builds.
Might be we only added this memset() to stop assertions that the node
wasn't already in another linked list?
Anyway, even ignoring all that, this reads a lot more clearly to me.
Change-Id: Iacfe507be2a53807caf85d30f4c4712d8f6e86dc
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/258449
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Jim Van Verth <jvanverth@google.com>
1 file changed