Revision: 76432
          http://sourceforge.net/p/brlcad/code/76432
Author:   starseeker
Date:     2020-07-22 21:25:56 +0000 (Wed, 22 Jul 2020)
Log Message:
-----------
Note reason for validation

Modified Paths:
--------------
    brlcad/branches/gedplugins/src/libged/exec_mapping.cpp

Modified: brlcad/branches/gedplugins/src/libged/exec_mapping.cpp
===================================================================
--- brlcad/branches/gedplugins/src/libged/exec_mapping.cpp      2020-07-22 
21:20:05 UTC (rev 76431)
+++ brlcad/branches/gedplugins/src/libged/exec_mapping.cpp      2020-07-22 
21:25:56 UTC (rev 76432)
@@ -19,9 +19,12 @@
  */
 /** @file exec_wrapping.cpp
  *
- * Provide compile time wrappers that pass specific libged function
- * calls through to the plugin system.
+ * Provide compile time wrappers that pass specific libged function calls
+ * through to the plugin system.
  *
+ * Some validation is also performed to ensure the argv[0] string value makes
+ * sense - unlike raw function calls, correct argv[0] values are important with
+ * ged_exec to ensure the expected functionality is invoked.
  */
 
 #include "common.h"
@@ -36,6 +39,7 @@
            int ret = ged_cmd_valid(argv[0], fname); \
            if (ret) { \
                bu_log("Error(%d): LIBGED command \"ged_" #x "\" called with an 
unexpected argv[0] value: %s\n", ret, argv[0]); \
+               return GED_UNKNOWN; \
            }\
            ret = ged_exec(gedp, argc, argv); \
            return ret; \

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to