Assorted `syntax-check' fixes.
[bpt/guile.git] / libguile / error.c
index 9968eb1..4b6bab8 100644 (file)
@@ -1,31 +1,34 @@
-/* Copyright (C) 1995,1996,1997,1998,2000,2001 Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,2000,2001, 2004, 2006, 2010 Free Software Foundation, Inc.
  * 
  * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
+ * modify it under the terms of the GNU Lesser General Public License
+ * as published by the Free Software Foundation; either version 3 of
+ * the License, or (at your option) any later version.
  *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * This library is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Lesser General Public License for more details.
  *
  * You should have received a copy of the GNU Lesser General Public
  * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+ * 02110-1301 USA
  */
 
 
 \f
 
-#if HAVE_CONFIG_H
+#ifdef HAVE_CONFIG_H
 #  include <config.h>
 #endif
 
+#include <stdlib.h>
 #include <stdio.h>
 #include <errno.h>
 
 #include "libguile/_scm.h"
+#include "libguile/dynwind.h"
 #include "libguile/pairs.h"
 #include "libguile/strings.h"
 #include "libguile/throw.h"
 #endif
 \f
 
-\f
 /* {Errors and Exceptional Conditions}
  */
 
 
-/* All errors should pass through here.  */
+/* Scheme interface to scm_error_scm.  */
 void
 scm_error (SCM key, const char *subr, const char *message, SCM args, SCM rest)
 {
-  SCM arg_list;
-  if (scm_gc_running_p)
-    {
-      /* The error occured during GC --- abort */
-      fprintf (stderr, "Error in %s during GC: %s\n",
-              subr ? subr : "unknown function",
-              message ? message : "<empty message>");
-      abort ();
-    }
-  arg_list = scm_list_4 (subr ? scm_makfrom0str (subr) : SCM_BOOL_F,
-                        message ? scm_makfrom0str (message) : SCM_BOOL_F,
-                        args,
-                        rest);
-  scm_ithrow (key, arg_list, 1);
-  
-  /* No return, but just in case: */
-  {
-    const char msg[] = "guile:scm_error:scm_ithrow returned!\n";
-
-    write (2, msg, (sizeof msg) - 1);
-  }
-  exit (1);
+  scm_error_scm 
+    (key,
+     (subr == NULL) ? SCM_BOOL_F : scm_from_locale_string (subr),
+     (message == NULL) ? SCM_BOOL_F : scm_from_locale_string (message),
+     args, rest);
 }
 
-/* Scheme interface to scm_error.  */
+/* All errors should pass through here.  */
 SCM_DEFINE (scm_error_scm, "scm-error", 5, 0, 0, 
            (SCM key, SCM subr, SCM message, SCM args, SCM data),
            "Raise an error with key @var{key}.  @var{subr} can be a string\n"
@@ -93,41 +78,24 @@ SCM_DEFINE (scm_error_scm, "scm-error", 5, 0, 0,
            "@code{#f} depending on @var{key}: if @var{key} is\n"
            "@code{system-error} then it should be a list containing the\n"
            "Unix @code{errno} value; If @var{key} is @code{signal} then it\n"
-           "should be a list containing the Unix signal number; otherwise\n"
+           "should be a list containing the Unix signal number; If\n"
+           "@var{key} is @code{out-of-range} or @code{wrong-type-arg},\n"
+            "it is a list containing the bad value; otherwise\n"
            "it will usually be @code{#f}.")
 #define FUNC_NAME s_scm_error_scm
 {
-  char *szSubr;
-  char *szMessage;
-
-  SCM_VALIDATE_SYMBOL (1, key);
-
-  if (scm_is_false (subr))
-    {
-      szSubr = NULL;
-    }
-  else if (SCM_SYMBOLP (subr))
-    {
-      szSubr = SCM_SYMBOL_CHARS (subr);
-    }
-  else
-    {
-      SCM_VALIDATE_STRING (2, subr);
-      szSubr = SCM_STRING_CHARS (subr);
-    }
-
-  if (scm_is_false (message))
-    {
-      szMessage = NULL;
-    }
-  else
+  if (scm_gc_running_p)
     {
-      SCM_VALIDATE_STRING (2, message);
-      szMessage = SCM_STRING_CHARS (message);
+      /* The error occured during GC --- abort */
+      fprintf (stderr, "Guile: error during GC.\n"),
+      abort ();
     }
 
-  scm_error (key, szSubr, szMessage, args, data);
-  /* not reached.  */
+  scm_ithrow (key, scm_list_4 (subr, message, args, data), 1);
+  
+  /* No return, but just in case: */
+  fprintf (stderr, "Guile scm_ithrow returned!\n");
+  exit (EXIT_FAILURE);
 }
 #undef FUNC_NAME
 
@@ -142,14 +110,35 @@ SCM_DEFINE (scm_error_scm, "scm-error", 5, 0, 0,
 # define SCM_I_ERRNO() errno
 #endif /* __MINGW32__ */
 
+/* strerror may not be thread safe, for instance in glibc (version 2.3.2) an
+   error number not among the known values results in a string like "Unknown
+   error 9999" formed in a static buffer, which will be overwritten by a
+   similar call in another thread.  A test program running two threads with
+   different unknown error numbers can trip this fairly quickly.
+
+   Some systems don't do what glibc does, instead just giving a single
+   "Unknown error" for unrecognised numbers.  It doesn't seem worth trying
+   to tell if that's the case, a mutex is reasonably fast, and strerror
+   isn't needed very often.
+
+   strerror_r (when available) could be used, it might be a touch faster
+   than a frame and a mutex, though there's probably not much
+   difference.  */
+
 SCM_DEFINE (scm_strerror, "strerror", 1, 0, 0, 
             (SCM err),
            "Return the Unix error message corresponding to @var{err}, which\n"
            "must be an integer value.")
 #define FUNC_NAME s_scm_strerror
 {
-  SCM_VALIDATE_INUM (1, err);
-  return scm_makfrom0str (SCM_I_STRERROR (SCM_INUM (err)));
+  SCM ret;
+  scm_dynwind_begin (0);
+  scm_i_dynwind_pthread_mutex_lock (&scm_i_misc_mutex);
+
+  ret = scm_from_locale_string (SCM_I_STRERROR (scm_to_int (err)));
+
+  scm_dynwind_end ();
+  return ret;
 }
 #undef FUNC_NAME
 
@@ -157,13 +146,12 @@ SCM_GLOBAL_SYMBOL (scm_system_error_key, "system-error");
 void
 scm_syserror (const char *subr)
 {
-  int save_errno = SCM_I_ERRNO ();
-  
+  SCM err = scm_from_int (SCM_I_ERRNO ());
   scm_error (scm_system_error_key,
             subr,
             "~A",
-            scm_cons (scm_makfrom0str (SCM_I_STRERROR (save_errno)), SCM_EOL),
-            scm_cons (scm_from_int (save_errno), SCM_EOL));
+            scm_cons (scm_strerror (err), SCM_EOL),
+            scm_cons (err, SCM_EOL));
 }
 
 void
@@ -193,9 +181,9 @@ scm_out_of_range (const char *subr, SCM bad_value)
 {
   scm_error (scm_out_of_range_key,
             subr,
-            "Argument out of range: ~S",
+            "Value out of range: ~S",
              scm_list_1 (bad_value),
-            SCM_BOOL_F);
+             scm_list_1 (bad_value));
 }
 
 void
@@ -203,9 +191,9 @@ scm_out_of_range_pos (const char *subr, SCM bad_value, SCM pos)
 {
   scm_error (scm_out_of_range_key,
             subr,
-            "Argument ~S out of range: ~S",
+            "Argument ~A out of range: ~S",
              scm_list_2 (pos, bad_value),
-            SCM_BOOL_F);
+            scm_list_1 (bad_value));
 }
 
 
@@ -227,7 +215,7 @@ scm_error_num_args_subr (const char *subr)
   scm_error (scm_args_number_key,
             NULL,
             "Wrong number of arguments to ~A",
-            scm_list_1 (scm_makfrom0str (subr)),
+            scm_list_1 (scm_from_locale_string (subr)),
             SCM_BOOL_F);
 }
 
@@ -238,29 +226,45 @@ scm_wrong_type_arg (const char *subr, int pos, SCM bad_value)
 {
   scm_error (scm_arg_type_key,
             subr,
-            (pos == 0) ? "Wrong type argument: ~S"
+            (pos == 0) ? "Wrong type: ~S"
             : "Wrong type argument in position ~A: ~S",
             (pos == 0) ? scm_list_1 (bad_value)
-            : scm_list_2 (SCM_I_MAKINUM (pos), bad_value),
-            SCM_BOOL_F);
+            : scm_list_2 (scm_from_int (pos), bad_value),
+            scm_list_1 (bad_value));
+}
+
+void
+scm_i_wrong_type_arg_symbol (SCM symbol, int pos, SCM bad_value)
+{
+  scm_error_scm (scm_arg_type_key,
+                scm_symbol_to_string (symbol),
+                (pos == 0) ? scm_from_locale_string ("Wrong type: ~S")
+                : scm_from_locale_string ("Wrong type argument in position ~A: ~S"),
+                (pos == 0) ? scm_list_1 (bad_value)
+                : scm_list_2 (scm_from_int (pos), bad_value),
+                scm_list_1 (bad_value));
+  scm_remember_upto_here_2 (symbol, bad_value);
 }
 
 void
 scm_wrong_type_arg_msg (const char *subr, int pos, SCM bad_value, const char *szMessage)
 {
-  SCM msg = scm_makfrom0str(szMessage);
-  if (pos == 0) {
-    scm_error (scm_arg_type_key,
-               subr, "Wrong type argument (expecting ~A): ~S",
-               scm_list_2 (msg, bad_value),
-               SCM_BOOL_F);
-  } else {
-    scm_error (scm_arg_type_key,
-               subr,
-               "Wrong type argument in position ~A (expecting ~A): ~S",
-               scm_list_3 (SCM_I_MAKINUM (pos), msg, bad_value),
-               SCM_BOOL_F);
-  }
+  SCM msg = scm_from_locale_string (szMessage);
+  if (pos == 0)
+    {
+      scm_error (scm_arg_type_key,
+                subr, "Wrong type (expecting ~A): ~S",
+                scm_list_2 (msg, bad_value),
+                scm_list_1 (bad_value));
+    }
+  else
+    {
+      scm_error (scm_arg_type_key,
+                subr,
+                "Wrong type argument in position ~A (expecting ~A): ~S",
+                scm_list_3 (scm_from_int (pos), msg, bad_value),
+                scm_list_1 (bad_value));
+    }
 }
 
 
@@ -268,11 +272,8 @@ SCM_GLOBAL_SYMBOL (scm_memory_alloc_key, "memory-allocation-error");
 void
 scm_memory_error (const char *subr)
 {
-  scm_error (scm_memory_alloc_key,
-            subr,
-            "Memory allocation error",
-            SCM_BOOL_F,
-            SCM_BOOL_F);
+  fprintf (stderr, "FATAL: memory error in %s\n", subr);
+  abort ();
 }
 
 SCM_GLOBAL_SYMBOL (scm_misc_error_key, "misc-error");
@@ -285,7 +286,7 @@ scm_misc_error (const char *subr, const char *message, SCM args)
 void
 scm_init_error ()
 {
-#include "libguile/cpp_err_symbols.c"
+#include "libguile/cpp-E.c"
 #include "libguile/error.x"
 }