search

SpatioTemporal / STARE

SpatioTemporal Adaptive Resolution Encoding, a unifying index for geo-located data.
Other
10 stars 7 forks source link

STARE should not just exit. Need a better exception strategy. #75

Closed michaelleerilee closed 3 years ago

jgallagher59701 commented 3 years ago

Yes. A library should never just exit. It would be good if the C++ library defined an error exception (using std::exception as the parent class) and used that to signal errors. You could define several classes if needed, once for a warning and one for a fatal error, for example. Either way, they should all inherit from std::exception.

On Mar 3, 2021, at 21:59, Rilee notifications@github.com wrote:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/SpatioTemporal/STARE/issues/75, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB7Q4KUATP5DDMQM7UQGAKLTB4HRJANCNFSM4YSQ4DVQ.

-- James Gallagher jgallagher@opendap.org

michaelleerilee commented 3 years ago

Added SpatialFailures instead of exits in 0.16.4.