scipopt / russcip

Rust interface for SCIP
https://crates.io/crates/russcip
Apache License 2.0
34 stars 11 forks source link

error handling with Result #10

Closed lovasoa closed 1 year ago

lovasoa commented 1 year ago

It looks like throughout the library, errors are handled with panics (which crash the user's program). This is probably not what the library user wants. Shouldn't scip errors be translated to rust Results instead, so that they can be handled with traditional rust error handling ?

https://github.com/mmghannam/russcip/blob/cdda8e7142d7bf62769eafc5cd58fd27eb305045/src/lib.rs#L56-L61