Changeset: 64d9b63de142 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=64d9b63de142
Modified Files:
monetdb5/mal/mal_interpreter.c
Branch: Jul2017
Log Message:
Fix thinko/typo.
In
getArgName(mb, l, j) || strcmp(getArgName(mb, l, j), "ANYexception") == 0
if getArgName(mb, l, j) == NULL (which, by the way, it cannot be), the
strcmp is called with a NULL first argument. Compilers may complain
about this. And since the first part cannot be NULL, the second was
actually never called.
diffs (13 lines):
diff --git a/monetdb5/mal/mal_interpreter.c b/monetdb5/mal/mal_interpreter.c
--- a/monetdb5/mal/mal_interpreter.c
+++ b/monetdb5/mal/mal_interpreter.c
@@ -950,8 +950,7 @@ str runMALsequence(Client cntxt, MalBlkP
for (j = 0; j < l->retc; j++)
if (getArg(l, j) ==
exceptionVar)
break;
- else if (getArgName(mb, l, j) ||
-
strcmp(getArgName(mb, l, j), "ANYexception") == 0)
+ else if (strcmp(getArgName(mb,
l, j), "ANYexception") == 0)
break;
if (j < l->retc)
break;
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list
