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]

https://github.com/google/s2geometry/pull/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