connormanning / entwine

Entwine - point cloud organization for massive datasets
https://entwine.io
Other
451 stars 128 forks source link

Changed exception handler from std::runtime_error to std::exception. #265

Closed tfiner closed 3 years ago

tfiner commented 3 years ago

I added an exception handler for the base class of std::exception: this will future proof all other std::exception derived exceptions. It also fixes the problem with JSON parsing errors crashing entwine.

I also added the catch all, in case some future library doesn't derive from std::exception, with the thinking that It is far better to write something out on stdout/stderr than just crash. See https://github.com/connormanning/entwine/issues/264