mes: More informative error reporting for number asserts.
authorJan Nieuwenhuizen <janneke@gnu.org>
Thu, 25 Jan 2018 06:00:48 +0000 (07:00 +0100)
committerJan Nieuwenhuizen <janneke@gnu.org>
Thu, 25 Jan 2018 06:00:48 +0000 (07:00 +0100)
* src/math.c (assert_number): New function.  Update assert () users.

module/mes/type-0.mes
src/lib.c
src/math.c
src/mes.c

index 6b709cfec043fae909e954420f677447798b53dd..c98f8113d68c90e09c2587db6edeff5c305cc775 100644 (file)
@@ -63,7 +63,7 @@
 
 (define (char? x)
   (and (eq? (core:type x) <cell:char>)
-       (>= (char->integer x) 0)))
+       (> (char->integer x) -1)))
 
 (define (eof-object? x)
   (and (eq? (core:type x) <cell:char>)
index f7296c7edfe500f9268878bec5817578ab544abf..ec1702b033f6a556920113ceea32674f12dcbfaf 100644 (file)
--- a/src/lib.c
+++ b/src/lib.c
@@ -96,6 +96,8 @@ display_helper (SCM x, int cont, char* sep, int fd, int write_p)
             x = CDR (x);
             while (x != cell_nil && i++ < 10)
               {
+                g_depth = 1;
+                //display_helper (CAAR (x), 0, "", fd, write_p); fputs (" ", fd);
                 fdisplay_ (CAAR (x), fd, write_p); fputs (" ", fd);
                 x = CDR (x);
               }
index 1e9df4dac5119498e21bfdee95c5446a56dcaac3..4117d1aa174b4f2698bc641d32c5f8b9ad9596e8 100644 (file)
@@ -1,6 +1,6 @@
 /* -*-comment-start: "//";comment-end:""-*-
  * Mes --- Maxwell Equations of Software
- * Copyright © 2016,2017 Jan Nieuwenhuizen <janneke@gnu.org>
+ * Copyright © 2016,2017,2018 Jan Nieuwenhuizen <janneke@gnu.org>
  *
  * This file is part of Mes.
  *
 
 #include <limits.h>
 
+void
+assert_number (char const* name, SCM x)
+{
+  if (TYPE (x) != TNUMBER)
+    {
+      eputs (name);
+      error (cell_symbol_not_a_number, x);
+    }
+}
+
 SCM
 greater_p (SCM x) ///((name . ">") (arity . n))
 {
   int n = INT_MAX;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("greater_p", CAR (x));
       if (VALUE (car (x)) >= n) return cell_f;
       n = VALUE (car (x));
       x = cdr (x);
@@ -40,7 +50,7 @@ less_p (SCM x) ///((name . "<") (arity . n))
   int n = INT_MIN;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("less_p", CAR (x));
       if (VALUE (car (x)) <= n) return cell_f;
       n = VALUE (car (x));
       x = cdr (x);
@@ -52,8 +62,8 @@ SCM
 is_p (SCM x) ///((name . "=") (arity . n))
 {
   if (x == cell_nil) return cell_t;
-  assert (TYPE (car (x)) == TNUMBER);
-  int n = VALUE (car (x));
+  assert_number ("is_p", CAR (x));
+  int n = VALUE (CAR (x));
   x = cdr (x);
   while (x != cell_nil)
     {
@@ -66,15 +76,15 @@ is_p (SCM x) ///((name . "=") (arity . n))
 SCM
 minus (SCM x) ///((name . "-") (arity . n))
 {
-  SCM a = car (x);
-  assert (TYPE (a) == TNUMBER);
+  assert_number ("minus", CAR (x));
+  SCM a = CAR (x);
   int n = VALUE (a);
   x = cdr (x);
   if (x == cell_nil)
     n = -n;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("minus", CAR (x));
       n -= VALUE (car (x));
       x = cdr (x);
     }
@@ -87,7 +97,7 @@ plus (SCM x) ///((name . "+") (arity . n))
   int n = 0;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("plus", CAR (x));
       n += VALUE (car (x));
       x = cdr (x);
     }
@@ -99,13 +109,13 @@ divide (SCM x) ///((name . "/") (arity . n))
 {
   int n = 1;
   if (x != cell_nil) {
-    assert (TYPE (car (x)) == TNUMBER);
+    assert_number ("divide", CAR (x));
     n = VALUE (car (x));
     x = cdr (x);
   }
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("divide", CAR (x));
       n /= VALUE (car (x));
       x = cdr (x);
     }
@@ -115,8 +125,8 @@ divide (SCM x) ///((name . "/") (arity . n))
 SCM
 modulo (SCM a, SCM b)
 {
-  assert (TYPE (a) == TNUMBER);
-  assert (TYPE (b) == TNUMBER);
+  assert_number ("modulo", a);
+  assert_number ("modulo", b);
   int x = VALUE (a);
   while (x < 0) x += VALUE (b);
   return MAKE_NUMBER (x % VALUE (b));
@@ -128,7 +138,7 @@ multiply (SCM x) ///((name . "*") (arity . n))
   int n = 1;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("multiply", CAR (x));
       n *= VALUE (car (x));
       x = cdr (x);
     }
@@ -141,7 +151,7 @@ logand (SCM x) ///((arity . n))
   int n = 0;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("multiply", CAR (x));
       n &= VALUE (car (x));
       x = cdr (x);
     }
@@ -154,7 +164,7 @@ logior (SCM x) ///((arity . n))
   int n = 0;
   while (x != cell_nil)
     {
-      assert (TYPE (car (x)) == TNUMBER);
+      assert_number ("logior", CAR (x));
       n |= VALUE (car (x));
       x = cdr (x);
     }
@@ -164,7 +174,7 @@ logior (SCM x) ///((arity . n))
 SCM
 lognot (SCM x)
 {
-  assert (TYPE (x) == TNUMBER);
+  assert_number ("lognot", x);
   int n = ~VALUE (x);
   return MAKE_NUMBER (n);
 }
@@ -172,8 +182,8 @@ lognot (SCM x)
 SCM
 ash (SCM n, SCM count)
 {
-  assert (TYPE (n) == TNUMBER);
-  assert (TYPE (count) == TNUMBER);
+  assert_number ("ash", n);
+  assert_number ("ash", count);
   int cn = VALUE (n);
   int ccount = VALUE (count);
   return MAKE_NUMBER ((ccount < 0) ? cn >> -ccount : cn << ccount);
index 603ace0c674100eb31ed156181ddff1c6306eff7..743e298fe5303b01d7e1b510e82c5564bb29a1e4 100644 (file)
--- a/src/mes.c
+++ b/src/mes.c
@@ -160,6 +160,7 @@ struct scm scm_symbol_write = {TSYMBOL, "write",0};
 struct scm scm_symbol_display = {TSYMBOL, "display",0};
 
 struct scm scm_symbol_throw = {TSYMBOL, "throw",0};
+struct scm scm_symbol_not_a_number = {TSYMBOL, "not-a-number",0};
 struct scm scm_symbol_not_a_pair = {TSYMBOL, "not-a-pair",0};
 struct scm scm_symbol_system_error = {TSYMBOL, "system-error",0};
 struct scm scm_symbol_wrong_number_of_args = {TSYMBOL, "wrong-number-of-args",0};