scipopt / russcip

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

memory can be freed while keeping a rust reference to it #9

Closed lovasoa closed 1 year ago

lovasoa commented 1 year ago

https://github.com/mmghannam/russcip/blob/cdda8e7142d7bf62769eafc5cd58fd27eb305045/src/model.rs#L193-L195

Souldn't this be a Drop implementation ?

The library should not leak memory, or free memory that can still be accessed. Maybe you should add a test where you solve many large problems in a loop ?