[Xotcl] bug in XOTcl_DeleteCommandFromToken ?

Jeff Hobbs jeffh at ActiveState.com
Tue Nov 30 20:57:22 CET 2004


I found this oddity in XOTcl_DeleteCommandFromToken:

   XOTclCallStackContent *csc = cs->top;
 
   for (; csc > cs->content; csc--) {
-    if (csc->cmdPtr == cmd)
-      csc->cmdPtr = NULL;
-    break;
+      if (csc->cmdPtr == cmd) {
+	  csc->cmdPtr = NULL;
+	  break;
+      }
   }
   return Tcl_DeleteCommandFromToken(in, cmd);
 }

Is my fix correct, or is there some magic reason why you
would want to break after the first 'if' check no matter
what?

BTW - any conditionals without { } is BAD CODE, for exactly
the above reason.

  Jeff Hobbs, The Tcl Guy
  http://www.ActiveState.com/, a division of Sophos



More information about the Xotcl mailing list