Don't fold argument check into expression.
authorChris Hanson <org/chris-hanson/cph>
Sat, 25 Feb 2012 06:28:34 +0000 (22:28 -0800)
committerChris Hanson <org/chris-hanson/cph>
Sat, 25 Feb 2012 06:28:34 +0000 (22:28 -0800)
src/microcode/fixnum.c

index b889bf897a72976de02d3db2c600e5e821f9a94d..80f64263c1b6d84176cccf093076179934300627 100644 (file)
@@ -209,9 +209,9 @@ DEFINE_PRIMITIVE ("FIXNUM-QUOTIENT", Prim_fixnum_quotient, 2, 2, 0)
   {
     long numerator = (arg_fixnum (1));
     long denominator = (arg_fixnum (2));
-    FIXNUM_RESULT (denominator == 0
-                  ? (error_bad_range_arg (2), 0)
-                  : FIXNUM_QUOTIENT (numerator, denominator));
+    if (denominator == 0)
+      error_bad_range_arg (2);
+    FIXNUM_RESULT (FIXNUM_QUOTIENT (numerator, denominator));
   }
 }
 
@@ -221,9 +221,9 @@ DEFINE_PRIMITIVE ("FIXNUM-REMAINDER", Prim_fixnum_remainder, 2, 2, 0)
   {
     long numerator = (arg_fixnum (1));
     long denominator = (arg_fixnum (2));
-    FIXNUM_RESULT (denominator == 0
-                  ? (error_bad_range_arg (2), 0)
-                  : FIXNUM_REMAINDER (numerator, denominator));
+    if (denominator == 0)
+      error_bad_range_arg (2);
+    FIXNUM_RESULT (FIXNUM_REMAINDER (numerator, denominator));
   }
 }