*** empty log message ***
[bpt/guile.git] / libguile / read.c
index 51e197d..5edfcf7 100644 (file)
  * If you write modifications of your own for GUILE, it is your choice
  * whether to permit this exception to apply to your modifications.
  * If you do not wish that, delete this exception notice.  */
+
+/* Software engineering face-lift by Greg J. Badros, 11-Dec-1999,
+   gjb@cs.washington.edu, http://www.cs.washington.edu/homes/gjb */
+
 \f
 
 #include <stdio.h>
@@ -52,6 +56,7 @@
 #include "hashtab.h"
 #include "hash.h"
 
+#include "validate.h"
 #include "read.h"
 
 \f
@@ -65,43 +70,39 @@ scm_option scm_read_opts[] = {
     "Record positions of source code expressions." },
   { SCM_OPTION_BOOLEAN, "case-insensitive", 0,
     "Convert symbols to lower case."},
-  { SCM_OPTION_SCM, "keywords", SCM_BOOL_F,
+  { SCM_OPTION_SCM, "keywords", SCM_UNPACK (SCM_BOOL_F),
     "Style of keyword recognition: #f or 'prefix"}
 };
 
-SCM_PROC (s_read_options, "read-options-interface", 0, 1, 0, scm_read_options);
-
-SCM
-scm_read_options (setting)
-     SCM setting;
+SCM_DEFINE (scm_read_options, "read-options-interface", 0, 1, 0, 
+            (SCM setting),
+"")
+#define FUNC_NAME s_scm_read_options
 {
   SCM ans = scm_options (setting,
                         scm_read_opts,
                         SCM_N_READ_OPTIONS,
-                        s_read_options);
+                        FUNC_NAME);
   if (SCM_COPY_SOURCE_P)
     SCM_RECORD_POSITIONS_P = 1;
   return ans;
 }
+#undef FUNC_NAME
 
 /* An association list mapping extra hash characters to procedures.  */
 static SCM *scm_read_hash_procedures;
 
-SCM_PROC (s_read, "read", 0, 1, 0, scm_read);
-
-SCM 
-scm_read (port)
-     SCM port;
+SCM_DEFINE (scm_read, "read", 0, 1, 0, 
+            (SCM port),
+"")
+#define FUNC_NAME s_scm_read
 {
   int c;
   SCM tok_buf, copy;
 
   if (SCM_UNBNDP (port))
     port = scm_cur_inp;
-  SCM_ASSERT (SCM_NIMP (port) && SCM_OPINPORTP (port),
-             port,
-             SCM_ARG1,
-             s_read);
+  SCM_VALIDATE_OPINPORT (1,port);
 
   c = scm_flush_ws (port, (char *) NULL);
   if (EOF == c)
@@ -111,12 +112,12 @@ scm_read (port)
   tok_buf = scm_makstr (30L, 0);
   return scm_lreadr (&tok_buf, port, &copy);
 }
+#undef FUNC_NAME
 
 
 
 char *
-scm_grow_tok_buf (tok_buf)
-     SCM * tok_buf;
+scm_grow_tok_buf (SCM *tok_buf)
 {
   scm_vector_set_length_x (*tok_buf, SCM_MAKINUM (2 * SCM_LENGTH (*tok_buf)));
   return SCM_CHARS (*tok_buf);
@@ -125,9 +126,7 @@ scm_grow_tok_buf (tok_buf)
 
 
 int 
-scm_flush_ws (port, eoferr)
-     SCM port;
-     const char *eoferr;
+scm_flush_ws (SCM port, const char *eoferr)
 {
   register int c;
   while (1)
@@ -162,9 +161,7 @@ scm_flush_ws (port, eoferr)
 
 
 int
-scm_casei_streq (s1, s2)
-     char * s1;
-     char * s2;
+scm_casei_streq (char *s1, char *s2)
 {
   while (*s1 && *s2)
     if (scm_downcase((int)*s1) != scm_downcase((int)*s2))
@@ -184,14 +181,8 @@ scm_casei_streq (s1, s2)
 #ifndef DEBUG_EXTENSIONS
 #define recsexpr(obj, line, column, filename) (obj)
 #else
-static SCM recsexpr SCM_P ((SCM obj, int line, int column, SCM filename));
-
 static SCM
-recsexpr (obj, line, column, filename)
-     SCM obj;
-     int line;
-     int column;
-     SCM filename;
+recsexpr (SCM obj,int line,int column,SCM filename)
 {
   if (SCM_IMP (obj) || SCM_NCONSP(obj))
     return obj;
@@ -206,7 +197,7 @@ recsexpr (obj, line, column, filename)
          {
            copy = scm_cons (recsexpr (SCM_CAR (obj), line, column, filename),
                             SCM_UNDEFINED);
-           while (SCM_NIMP (tmp = SCM_CDR (tmp)) && SCM_CONSP (tmp))
+           while ((tmp = SCM_CDR (tmp)) && SCM_CONSP (tmp))
              {
                SCM_SETCDR (copy, scm_cons (recsexpr (SCM_CAR (tmp),
                                                      line,
@@ -220,7 +211,7 @@ recsexpr (obj, line, column, filename)
        else
          {
            recsexpr (SCM_CAR (obj), line, column, filename);
-           while (SCM_NIMP (tmp = SCM_CDR (tmp)) && SCM_CONSP (tmp))
+           while ((tmp = SCM_CDR (tmp)) && SCM_CONSP (tmp))
              recsexpr (SCM_CAR (tmp), line, column, filename);
            copy = SCM_UNDEFINED;
          }
@@ -242,8 +233,7 @@ recsexpr (obj, line, column, filename)
    newline/exclamation-point/sharp-sign/newline sequence.  */
 
 static void
-skip_scsh_block_comment (port)
-     SCM port;
+skip_scsh_block_comment (SCM port)
 {
   /* Is this portable?  Dear God, spare me from the non-eight-bit
      characters.  But is it tasteful?  */
@@ -264,23 +254,19 @@ skip_scsh_block_comment (port)
     }
 }
 
-static SCM
-scm_get_hash_procedure SCM_P ((int c));
+static SCM scm_get_hash_procedure(int c);
 
 static char s_list[]="list";
 
 SCM 
-scm_lreadr (tok_buf, port, copy)
-     SCM *tok_buf;
-     SCM port;
-     SCM *copy;
+scm_lreadr (SCM *tok_buf,SCM port,SCM *copy)
 {
   int c;
   scm_sizet j;
   SCM p;
                                  
 tryagain:
-  c = scm_flush_ws (port, s_read);
+  c = scm_flush_ws (port, s_scm_read);
 tryagain_no_flush_ws:
   switch (c)
     {
@@ -385,17 +371,17 @@ tryagain_no_flush_ws:
          c = scm_getc (port);
          j = scm_read_token (c, tok_buf, port, 0);
          if (j == 1)
-           return SCM_MAKICHR (c);
+           return SCM_MAKE_CHAR (c);
          if (c >= '0' && c < '8')
            {
              p = scm_istr2int (SCM_CHARS (*tok_buf), (long) j, 8);
              if (SCM_NFALSEP (p))
-               return SCM_MAKICHR (SCM_INUM (p));
+               return SCM_MAKE_CHAR (SCM_INUM (p));
            }
          for (c = 0; c < scm_n_charnames; c++)
            if (scm_charnames[c]
                && (scm_casei_streq (scm_charnames[c], SCM_CHARS (*tok_buf))))
-             return SCM_MAKICHR (scm_charnums[c]);
+             return SCM_MAKE_CHAR (scm_charnums[c]);
          scm_wta (SCM_UNDEFINED, "unknown # object: #\\", SCM_CHARS (*tok_buf));
 
          /* #:SYMBOL is a syntax for keywords supported in all contexts.  */
@@ -416,7 +402,7 @@ tryagain_no_flush_ws:
                SCM got;
 
                got = scm_apply (sharp,
-                                SCM_MAKICHR (c),
+                                SCM_MAKE_CHAR (c),
                                 scm_acons (port, SCM_EOL, SCM_EOL));
                if (SCM_UNSPECIFIED == got)
                  goto unkshrp;
@@ -428,8 +414,8 @@ tryagain_no_flush_ws:
              }
          }
        unkshrp:
-         scm_misc_error (s_read, "Unknown # object: %S",
-                         scm_listify (SCM_MAKICHR (c), SCM_UNDEFINED));
+         scm_misc_error (s_scm_read, "Unknown # object: ~S",
+                         scm_listify (SCM_MAKE_CHAR (c), SCM_UNDEFINED));
        }
 
     case '"':
@@ -503,7 +489,7 @@ tryagain_no_flush_ws:
       goto tok;
 
     case ':':
-      if (SCM_KEYWORD_STYLE == scm_keyword_prefix)
+      if (SCM_PACK (SCM_KEYWORD_STYLE) == scm_keyword_prefix)
        {
          j = scm_read_token ('-', tok_buf, port, 0);
          p = scm_intern (SCM_CHARS (*tok_buf), j);
@@ -525,11 +511,7 @@ _Pragma ("noopt");         /* # pragma _CRI noopt */
 #endif
 
 scm_sizet 
-scm_read_token (ic, tok_buf, port, weird)
-     int ic;
-     SCM *tok_buf;
-     SCM port;
-     int weird;
+scm_read_token (int ic, SCM *tok_buf, SCM port, int weird)
 {
   register scm_sizet j;
   register int c;
@@ -615,11 +597,7 @@ _Pragma ("opt");           /* # pragma _CRI opt */
 #endif
 
 SCM 
-scm_lreadparen (tok_buf, port, name, copy)
-     SCM *tok_buf;
-     SCM port;
-     char *name;
-     SCM *copy;
+scm_lreadparen (SCM *tok_buf, SCM port, char *name, SCM *copy)
 {
   SCM tmp;
   SCM tl;
@@ -655,11 +633,7 @@ scm_lreadparen (tok_buf, port, name, copy)
 
 
 SCM 
-scm_lreadrecparen (tok_buf, port, name, copy)
-     SCM *tok_buf;
-     SCM port;
-     char *name;
-     SCM *copy;
+scm_lreadrecparen (SCM *tok_buf, SCM port, char *name, SCM *copy)
 {
   register int c;
   register SCM tmp;
@@ -683,7 +657,7 @@ scm_lreadrecparen (tok_buf, port, name, copy)
   /* Build the head of the list structure. */
   ans = tl = scm_cons (tmp, SCM_EOL);
   if (SCM_COPY_SOURCE_P)
-    ans2 = tl2 = scm_cons (SCM_NIMP (tmp) && SCM_CONSP (tmp)
+    ans2 = tl2 = scm_cons (SCM_CONSP (tmp)
                           ? *copy
                           : tmp,
                           SCM_EOL);
@@ -694,7 +668,7 @@ scm_lreadrecparen (tok_buf, port, name, copy)
        {
          SCM_SETCDR (tl, tmp = scm_lreadr (tok_buf, port, copy));
          if (SCM_COPY_SOURCE_P)
-           SCM_SETCDR (tl2, scm_cons (SCM_NIMP (tmp) && SCM_CONSP (tmp)
+           SCM_SETCDR (tl2, scm_cons (SCM_CONSP (tmp)
                                       ? *copy
                                       : tmp,
                                       SCM_EOL));
@@ -704,7 +678,7 @@ scm_lreadrecparen (tok_buf, port, name, copy)
        }
       tl = SCM_SETCDR (tl, scm_cons (tmp, SCM_EOL));
       if (SCM_COPY_SOURCE_P)
-       tl2 = SCM_SETCDR (tl2, scm_cons (SCM_NIMP (tmp) && SCM_CONSP (tmp)
+       tl2 = SCM_SETCDR (tl2, scm_cons (SCM_CONSP (tmp)
                                         ? *copy
                                         : tmp,
                                         SCM_EOL));
@@ -727,18 +701,17 @@ exit:
 
 /* Manipulate the read-hash-procedures alist.  This could be written in
    Scheme, but maybe it will also be used by C code during initialisation.  */
-SCM_PROC (s_read_hash_extend, "read-hash-extend", 2, 0, 0, scm_read_hash_extend);
-SCM
-scm_read_hash_extend (chr, proc)
-     SCM chr;
-     SCM proc;
+SCM_DEFINE (scm_read_hash_extend, "read-hash-extend", 2, 0, 0,
+            (SCM chr, SCM proc),
+"")
+#define FUNC_NAME s_scm_read_hash_extend
 {
   SCM this;
   SCM prev;
 
-  SCM_ASSERT (SCM_ICHRP(chr), chr, SCM_ARG1, s_read_hash_extend);
+  SCM_VALIDATE_CHAR (1,chr);
   SCM_ASSERT (SCM_FALSEP (proc) || SCM_NIMP(proc), proc, SCM_ARG2,
-             s_read_hash_extend);
+             FUNC_NAME);
 
   /* Check if chr is already in the alist.  */
   this = *scm_read_hash_procedures;
@@ -782,11 +755,11 @@ scm_read_hash_extend (chr, proc)
 
   return SCM_UNSPECIFIED;
 }
+#undef FUNC_NAME
 
 /* Recover the read-hash procedure corresponding to char c.  */
 static SCM
-scm_get_hash_procedure (c)
-     int c;
+scm_get_hash_procedure (int c)
 {
   SCM rest = *scm_read_hash_procedures;
 
@@ -795,7 +768,7 @@ scm_get_hash_procedure (c)
       if (SCM_NULLP (rest))
        return SCM_BOOL_F;
   
-      if (SCM_ICHR (SCM_CAAR (rest)) == c)
+      if (SCM_CHAR (SCM_CAAR (rest)) == c)
        return SCM_CDAR (rest);
      
       rest = SCM_CDR (rest);