geoarrow / geoarrow-c

Experimental C and C++ implementation of the GeoArrow specification
http://geoarrow.org/geoarrow-c/
Apache License 2.0
23 stars 3 forks source link

Clean up ArrowErrorSet() usage #30

Closed paleolimbot closed 1 year ago

paleolimbot commented 1 year ago

Provide our own GeoArrowErrorSet() for GeoArrowError to minimize the number of times we have to cast between error types.

codecov[bot] commented 1 year ago

Codecov Report

Merging #30 (4fc3f01) into main (aad3d68) will decrease coverage by 0.08%. The diff coverage is 70.27%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
- Coverage   91.47%   91.39%   -0.08%     
==========================================
  Files          23       24       +1     
  Lines        3588     3604      +16     
  Branches       59       59              
==========================================
+ Hits         3282     3294      +12     
- Misses        272      276       +4     
  Partials       34       34              
Impacted Files Coverage Δ
src/geoarrow/builder.c 89.91% <0.00%> (ø)
src/geoarrow/geoarrow.h 77.35% <ø> (ø)
src/geoarrow/kernel.c 91.90% <0.00%> (ø)
src/geoarrow/array_view.c 94.44% <50.00%> (ø)
src/geoarrow/wkt_reader.c 96.15% <50.00%> (ø)
src/geoarrow/util.c 75.00% <75.00%> (ø)
src/geoarrow/metadata.c 86.18% <100.00%> (ø)
src/geoarrow/wkb_reader.c 98.46% <100.00%> (ø)
src/geoarrow/wkt_writer.c 97.80% <100.00%> (ø)