mv warning from finalizer to freemodel!

 - that way it only covers the case of a user trying to double-free a
   model

Author: rschwarz

src/mpb_interface.jl

@@ -19,8 +19,12 @@ numvar(m::SCIPMathProgModel) = _getNumVars(m)
 numconstr(m::SCIPMathProgModel) = _getNumConss(m)
 
 function freemodel!(m::SCIPMathProgModel)
-    # call finalizer directly
-    freescip(m.inner)
+    if m.inner.ptr_model != C_NULL
+        # call finalizer directly
+        freescip(m.inner)
+    else
+        Base.warn_once("Tried to free already freed model, ignoring.")
+    end
 end
 
 # TODO: mapping for :SemiCont, :SemiInt

src/types.jl

@@ -25,8 +25,6 @@ function freescip(m::SCIPModel)
     if m.ptr_model != C_NULL
         _freeModel(m)
         m.ptr_model = C_NULL
-    else
-        Base.warn_once("Tried to free already freed model, ignoring.")
     end
 end