Undefined sanitizer issue

[MBkkt]

-fsanitize=nullability-arg: Passing null as a function parameter which is annotated with _Nonnull.

https://github.com/google/s2geometry/blob/master/src/s2/util/gtl/compact_array.h#L416

You can add if (old_capacity != 0)

Also if it something like vector with SSO, maybe better to use absl::InlinedVector?

[MBkkt]

#284

[jmr]

Also if it something like vector with SSO, maybe better to use absl::InlinedVector?

CompactArray also has space optimizations, hence the "Compact".

[MBkkt]

Ok