SSoelvsten / cal

University of California BDD Package
https://ssoelvsten.github.io/cal/
Other
2 stars 2 forks source link

Add moving ownership of Cal BDD Manager #19

Closed SSoelvsten closed 1 year ago

SSoelvsten commented 1 year ago

One could also implement the same by using the std::unique_ptr<Cal_BddManagerStruct, int(*)(Cal_BddManager)> but this...

  1. ... is simpler and makes for slightly cleaner code (there is no need to add a .get() everywhere.
  2. ... keeps the dependency on the std as small as possible. In fact, we may want to also eliminate the dependency on the std::vector<_>? This does not really create any issues, since calObj.hh is header-only and id always compiled by the user; hence, one does not run into any issues with different versions of the std library.