diff src/bytecode.c @ 4978:f48bf636045f

Automatic merge
author Ben Wing <ben@xemacs.org>
date Fri, 05 Feb 2010 05:04:43 -0600
parents fe0d3106cc36
children 99f8ebc082d9 ed624ab64583 b5df3737028a
line wrap: on
line diff
--- a/src/bytecode.c	Fri Feb 05 04:28:08 2010 -0600
+++ b/src/bytecode.c	Fri Feb 05 05:04:43 2010 -0600
@@ -121,8 +121,14 @@
 
 #ifndef ERROR_CHECK_BYTE_CODE
 
-# define bytecode_assert(x) disabled_assert (x)
-# define bytecode_assert_with_message(x, msg) disabled_assert(x)
+/* Normally we would use `x' instead of `0' in the argument list, to avoid
+   problems if `x' (an expression) has side effects, and warnings if `x'
+   contains variables or parameters that are otherwise unused.  But in
+   this case `x' contains references to vars and params that exist only
+   when ERROR_CHECK_BYTE_CODE, and leaving in `x' would result in compile
+   errors. */
+# define bytecode_assert(x) disabled_assert (0)
+# define bytecode_assert_with_message(x, msg) disabled_assert(0)
 # define bytecode_abort_with_message(msg) abort_with_message (msg)
 
 #else /* ERROR_CHECK_BYTE_CODE */