vfx-rs / openexr-bind

cppmm bindings for OpenEXR
BSD 3-Clause "New" or "Revised" License
11 stars 6 forks source link

Make sure all exceptions from cppmm are handled #27

Open scott-wilson opened 3 years ago

scott-wilson commented 3 years ago

We ran into an issue yesterday where an error wasn't handled in the safe API, causing a segfault. In order to avoid those for now, we should make sure that all functions from cppmm are handled. Even if it is just sys::Imf_func(...).into_result().unwrap().

The PR template should be updated to include that as a requirement for accepting a PR as well.