For #67:

- in Commands, do not "instrument" function call for member
function of base classes (including Commands and its parents),
instrument only actual commands.
- in autogui_commands.lua, detect invalid objects that became nil
when object was deleted in the meanwhile.

Author: BrunoLevy

lib/autogui_commands.lua

@@ -419,6 +419,9 @@ end
 function autogui.command_is_alive(cmd_state)
    if cmd_state.request_.object().is_a(OGF.Interface) then
       local grob = cmd_state.request_.object().grob
+      if grob == nil then
+	  return false
+      end
       if not grob.is_a(OGF.SceneGraph) and
          not scene_graph.is_bound(grob.name) then
          return false

lib/gizmo.lua

@@ -17,11 +17,6 @@ function gizmo_gui.draw_gizmo()
     end
     x0 = graphite_gui.left_pane_width
     y0 = 20.0
-
---  For testing camera centering (which is still buggy)
---  x0 = graphite_main_window.gx
---  y0 = graphite_main_window.gy
-
     L =  150.0
     ImOGuizmo.SetRect(x0,y0,L)
     ImOGuizmo.BeginFrame(false)

src/lib/OGF/scene_graph/commands/commands.cpp

@@ -84,6 +84,15 @@ namespace OGF {
         const ArgList& args_in, Any& ret_val
     ) {
 
+       // do not do anything special for functions inherited from
+       // Commands (and base classes).
+       if(
+	  ogf_meta<Commands>::meta_class()->find_method(method_name) !=
+	  nullptr
+       ) {
+	  return Interface::invoke_method(method_name, args_in, ret_val);
+       }
+
 	// override_lock_ is not used anywhere, except in a commented-out
 	// code in GeometricIntelligence plugin.