search

uber / h3

Hexagonal hierarchical geospatial indexing system
https://h3geo.org
Apache License 2.0
4.84k stars 459 forks source link

Fix possible signed integer overflow in ijToIjk #733

Closed isaacbrodsky closed 1 year ago

isaacbrodsky commented 1 year ago

Follow up to #706.

coveralls commented 1 year ago

Coverage Status

Coverage increased (+0.001%) to 98.642% when pulling 1309956604e9b7c0b69659385bf65b6dcdd92a34 on isaacbrodsky:fix-ijtoijk-signed-integer-overflow into 975de7a103ae8dfc22a629ef59890de283a5d8d8 on uber:master.

isaacbrodsky commented 1 year ago

CI seems to be failing due presumably unrelated error about E: Unable to locate package clang-format-9

isaacbrodsky commented 1 year ago

The fuzzer failure looks like an unrelated flaky fuzzer on polygonToCells

isaacbrodsky commented 1 year ago

Closing this in favor of using macros in #735.