* fports.c (scm_fdes_to_port): call fcntl F_GETFL to test that
[bpt/guile.git] / libguile / throw.c
index 9b28187..b3f9ed1 100644 (file)
@@ -1,4 +1,4 @@
-/*     Copyright (C) 1995,1996 Free Software Foundation, Inc.
+/*     Copyright (C) 1995, 1996, 1997, 1998 Free Software Foundation, Inc.
  * 
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -12,7 +12,8 @@
  * 
  * You should have received a copy of the GNU General Public License
  * along with this software; see the file COPYING.  If not, write to
- * the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.
+ * the Free Software Foundation, Inc., 59 Temple Place, Suite 330,
+ * Boston, MA 02111-1307 USA
  *
  * As a special exception, the Free Software Foundation gives permission
  * for additional uses of the text contained in its release of GUILE.
  *
  * 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.  
- */
+ * 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>
 #include "smob.h"
 #include "alist.h"
 #include "eval.h"
+#include "eq.h"
 #include "dynwind.h"
+#include "backtrace.h"
 #ifdef DEBUG_EXTENSIONS
 #include "debug.h"
 #endif
 #include "continuations.h"
+#include "stackchk.h"
+#include "stacks.h"
+#include "fluids.h"
 
+#include "validate.h"
 #include "throw.h"
-\f
-
 
-/* {Catch and Throw} 
- */
+\f
+/* the jump buffer data structure */
 static int scm_tc16_jmpbuffer;
 
-SCM scm_bad_throw_vcell;
+#define SCM_JMPBUFP(OBJ) (SCM_NIMP(OBJ) && (SCM_TYP16(OBJ) == scm_tc16_jmpbuffer))
 
-#define SCM_JMPBUFP(O) (SCM_TYP16(O) == scm_tc16_jmpbuffer)
-#define JBACTIVE(O) (SCM_CAR (O) & (1L << 16L))
-#define ACTIVATEJB(O)  (SCM_CAR (O) |= (1L << 16L))
-#define DEACTIVATEJB(O)  (SCM_CAR (O) &= ~(1L << 16L))
+#define JBACTIVE(OBJ) (SCM_UNPACK_CAR (OBJ) & (1L << 16L))
+#define ACTIVATEJB(OBJ)  (SCM_SETOR_CAR (OBJ, (1L << 16L)))
+#define DEACTIVATEJB(OBJ)  (SCM_SETAND_CAR (OBJ, ~(1L << 16L)))
 
 #ifndef DEBUG_EXTENSIONS
-#define JBJMPBUF(O) ((jmp_buf*)SCM_CDR (O) )
+#define JBJMPBUF(OBJ) ((jmp_buf*)SCM_CDR (OBJ) )
 #define SETJBJMPBUF SCM_SETCDR
 #else
-#define JBSCM_DFRAME(O) ((scm_debug_frame*)SCM_CAR (SCM_CDR (O)) )
-#define JBJMPBUF(O) ((jmp_buf*)SCM_CDR (SCM_CDR (O)) )
-#define SETJBSCM_DFRAME(O,X) SCM_CAR(SCM_CDR (O)) = (SCM)(X)
-#define SETJBJMPBUF(O,X) SCM_SETCDR(SCM_CDR (O), X)
+#define SCM_JBDFRAME(OBJ) ((scm_debug_frame*)SCM_CAR (SCM_CDR (OBJ)) )
+#define JBJMPBUF(OBJ) ((jmp_buf*)SCM_CDR (SCM_CDR (OBJ)) )
+#define SCM_SETJBDFRAME(OBJ,X) SCM_SETCAR (SCM_CDR (OBJ), (SCM)(X))
+#define SETJBJMPBUF(OBJ,X) SCM_SETCDR(SCM_CDR (OBJ), X)
 
-#ifdef __STDC__
 static scm_sizet
 freejb (SCM jbsmob)
-#else
-static scm_sizet
-freejb (jbsmob)
-     SCM jbsmob;
-#endif
 {
   scm_must_free ((char *) SCM_CDR (jbsmob));
   return sizeof (scm_cell);
 }
 #endif
 
-#ifdef __STDC__
-static int
-printjb (SCM exp, SCM port, int writing)
-#else
 static int
-printjb (exp, port, writing)
-     SCM exp;
-     SCM port;
-     int writing;
-#endif
+printjb (SCM exp, SCM port, scm_print_state *pstate)
 {
-  scm_gen_puts (scm_regular_string, "#<jmpbuffer ", port);
-  scm_gen_puts (scm_regular_string, JBACTIVE(exp) ? "(active) " : "(inactive) ", port);
-  scm_intprint((SCM) JBJMPBUF(exp), 16, port);
-  scm_gen_putc ('>', port);
+  scm_puts ("#<jmpbuffer ", port);
+  scm_puts (JBACTIVE(exp) ? "(active) " : "(inactive) ", port);
+  scm_intprint(SCM_UNPACK ( JBJMPBUF(exp) ), 16, port);
+
+  scm_putc ('>', port);
   return 1 ;
 }
 
-static scm_smobfuns jbsmob = {
-  scm_mark0,
-#ifdef DEBUG_EXTENSIONS
-  freejb,
-#else
-  scm_free0,
-#endif
-  printjb,
-  0
-};
 
-#ifdef __STDC__
 static SCM
 make_jmpbuf (void)
-#else
-static SCM
-make_jmpbuf ()
-#endif
 {
   SCM answer;
-  SCM_NEWCELL (answer);
-  SCM_DEFER_INTS;
+  SCM_REDEFER_INTS;
   {
 #ifdef DEBUG_EXTENSIONS
     char *mem = scm_must_malloc (sizeof (scm_cell), "jb");
-    SCM_SETCDR (answer, (SCM) mem);
 #endif
-    SCM_CAR(answer) = scm_tc16_jmpbuffer;
+#ifdef DEBUG_EXTENSIONS
+    SCM_NEWSMOB (answer, scm_tc16_jmpbuffer, mem);
+#else
+    SCM_NEWSMOB (answer, scm_tc16_jmpbuffer, 0);
+#endif
     SETJBJMPBUF(answer, (jmp_buf *)0);
     DEACTIVATEJB(answer);
   }
-  SCM_ALLOW_INTS;
+  SCM_REALLOW_INTS;
   return answer;
 }
 
+\f
+/* scm_internal_catch (the guts of catch) */
 
 struct jmp_buf_and_retval      /* use only on the stack, in scm_catch */
 {
@@ -151,174 +134,603 @@ struct jmp_buf_and_retval       /* use only on the stack, in scm_catch */
   SCM retval;
 };
 
-SCM_PROC(s_catch, "catch", 3, 0, 0, scm_catch);
-#ifdef __STDC__
-SCM
-scm_catch (SCM tag, SCM thunk, SCM handler)
-#else
+
+/* scm_internal_catch is the guts of catch.  It handles all the
+   mechanics of setting up a catch target, invoking the catch body,
+   and perhaps invoking the handler if the body does a throw.
+
+   The function is designed to be usable from C code, but is general
+   enough to implement all the semantics Guile Scheme expects from
+   throw.
+
+   TAG is the catch tag.  Typically, this is a symbol, but this
+   function doesn't actually care about that.
+
+   BODY is a pointer to a C function which runs the body of the catch;
+   this is the code you can throw from.  We call it like this:
+      BODY (BODY_DATA)
+   where:
+      BODY_DATA is just the BODY_DATA argument we received; we pass it
+        through to BODY as its first argument.  The caller can make
+        BODY_DATA point to anything useful that BODY might need.
+
+   HANDLER is a pointer to a C function to deal with a throw to TAG,
+   should one occur.  We call it like this:
+      HANDLER (HANDLER_DATA, THROWN_TAG, THROW_ARGS)
+   where
+      HANDLER_DATA is the HANDLER_DATA argument we recevied; it's the
+         same idea as BODY_DATA above.
+      THROWN_TAG is the tag that the user threw to; usually this is
+         TAG, but it could be something else if TAG was #t (i.e., a
+         catch-all), or the user threw to a jmpbuf.
+      THROW_ARGS is the list of arguments the user passed to the THROW
+         function, after the tag.
+
+   BODY_DATA is just a pointer we pass through to BODY.  HANDLER_DATA
+   is just a pointer we pass through to HANDLER.  We don't actually
+   use either of those pointers otherwise ourselves.  The idea is
+   that, if our caller wants to communicate something to BODY or
+   HANDLER, it can pass a pointer to it as MUMBLE_DATA, which BODY and
+   HANDLER can then use.  Think of it as a way to make BODY and
+   HANDLER closures, not just functions; MUMBLE_DATA points to the
+   enclosed variables.
+
+   Of course, it's up to the caller to make sure that any data a
+   MUMBLE_DATA needs is protected from GC.  A common way to do this is
+   to make MUMBLE_DATA a pointer to data stored in an automatic
+   structure variable; since the collector must scan the stack for
+   references anyway, this assures that any references in MUMBLE_DATA
+   will be found.  */
+
 SCM
-scm_catch (tag, thunk, handler)
-     SCM tag;
-     SCM thunk;
-     SCM handler;
-#endif
+scm_internal_catch (SCM tag, scm_catch_body_t body, void *body_data, scm_catch_handler_t handler, void *handler_data)
 {
   struct jmp_buf_and_retval jbr;
   SCM jmpbuf;
   SCM answer;
 
-  SCM_ASSERT ((tag == SCM_BOOL_F) || (SCM_NIMP(tag) && SCM_SYMBOLP(tag)) || (tag == SCM_BOOL_T),
-         tag, SCM_ARG1, s_catch);
   jmpbuf = make_jmpbuf ();
   answer = SCM_EOL;
   scm_dynwinds = scm_acons (tag, jmpbuf, scm_dynwinds);
   SETJBJMPBUF(jmpbuf, &jbr.buf);
 #ifdef DEBUG_EXTENSIONS
-  SETJBSCM_DFRAME(jmpbuf, last_debug_info_frame);
+  SCM_SETJBDFRAME(jmpbuf, scm_last_debug_frame);
 #endif
   if (setjmp (jbr.buf))
     {
       SCM throw_tag;
       SCM throw_args;
 
-      SCM_DEFER_INTS;
+#ifdef STACK_CHECKING
+      scm_stack_checking_enabled_p = SCM_STACK_CHECKING_P;
+#endif
+      SCM_REDEFER_INTS;
       DEACTIVATEJB (jmpbuf);
       scm_dynwinds = SCM_CDR (scm_dynwinds);
-      SCM_ALLOW_INTS;
+      SCM_REALLOW_INTS;
       throw_args = jbr.retval;
       throw_tag = jbr.throw_tag;
       jbr.throw_tag = SCM_EOL;
       jbr.retval = SCM_EOL;
-      answer = scm_apply (handler, scm_cons (throw_tag, throw_args), SCM_EOL);
+      answer = handler (handler_data, throw_tag, throw_args);
     }
   else
     {
       ACTIVATEJB (jmpbuf);
-      answer = scm_apply (thunk,
-                         ((tag == SCM_BOOL_F) ? scm_cons (jmpbuf, SCM_EOL) : SCM_EOL),
-                         SCM_EOL);
-      SCM_DEFER_INTS;
+      answer = body (body_data);
+      SCM_REDEFER_INTS;
       DEACTIVATEJB (jmpbuf);
       scm_dynwinds = SCM_CDR (scm_dynwinds);
-      SCM_ALLOW_INTS;
+      SCM_REALLOW_INTS;
     }
   return answer;
 }
 
 
-static char s_throw[];
-#ifdef __STDC__
+\f
+/* scm_internal_lazy_catch (the guts of lazy catching) */
+
+/* The smob tag for lazy_catch smobs.  */
+static long tc16_lazy_catch;
+
+/* This is the structure we put on the wind list for a lazy catch.  It
+   stores the handler function to call, and the data pointer to pass
+   through to it.  It's not a Scheme closure, but it is a function
+   with data, so the term "closure" is appropriate in its broader
+   sense.
+
+   (We don't need anything like this in the "eager" catch code,
+   because the same C frame runs both the body and the handler.)  */
+struct lazy_catch {
+  scm_catch_handler_t handler;
+  void *handler_data;
+};
+
+/* Strictly speaking, we could just pass a zero for our print
+   function, because we don't need to print them.  They should never
+   appear in normal data structures, only in the wind list.  However,
+   it might be nice for debugging someday... */
+static int
+print_lazy_catch (SCM closure, SCM port, scm_print_state *pstate)
+{
+  struct lazy_catch *c = (struct lazy_catch *) SCM_CDR (closure);
+  char buf[200];
+
+  sprintf (buf, "#<lazy-catch 0x%lx 0x%lx>",
+          (long) c->handler, (long) c->handler_data);
+  scm_puts (buf, port);
+
+  return 1;
+}
+
+
+/* Given a pointer to a lazy catch structure, return a smob for it,
+   suitable for inclusion in the wind list.  ("Ah yes, a Château
+   Gollombiere '72, non?").  */
+static SCM
+make_lazy_catch (struct lazy_catch *c)
+{
+  SCM_RETURN_NEWSMOB (tc16_lazy_catch, c);
+}
+
+#define SCM_LAZY_CATCH_P(obj) \
+  (SCM_NIMP (obj) && (SCM_UNPACK_CAR (obj) == tc16_lazy_catch))
+
+
+/* Exactly like scm_internal_catch, except:
+   - It does not unwind the stack (this is the major difference).
+   - If handler returns, its value is returned from the throw.  */
 SCM
-scm_ithrow (SCM key, SCM args, int noreturn)
-#else
+scm_internal_lazy_catch (SCM tag, scm_catch_body_t body, void *body_data, scm_catch_handler_t handler, void *handler_data)
+{
+  SCM lazy_catch, answer;
+  struct lazy_catch c;
+
+  c.handler = handler;
+  c.handler_data = handler_data;
+  lazy_catch = make_lazy_catch (&c);
+
+  SCM_REDEFER_INTS;
+  scm_dynwinds = scm_acons (tag, lazy_catch, scm_dynwinds);
+  SCM_REALLOW_INTS;
+
+  answer = (*body) (body_data);
+
+  SCM_REDEFER_INTS;
+  scm_dynwinds = SCM_CDR (scm_dynwinds);
+  SCM_REALLOW_INTS;
+
+  return answer;
+}
+
+\f
+/* scm_internal_stack_catch
+   Use this one if you want debugging information to be stored in
+   scm_the_last_stack_fluid on error. */
+
+static SCM
+ss_handler (void *data, SCM tag, SCM throw_args)
+{
+  /* Save the stack */
+  scm_fluid_set_x (SCM_CDR (scm_the_last_stack_fluid),
+                  scm_make_stack (scm_cons (SCM_BOOL_T, SCM_EOL)));
+  /* Throw the error */
+  return scm_throw (tag, throw_args);
+}
+
+struct cwss_data
+{
+  SCM tag;
+  scm_catch_body_t body;
+  void *data;
+};
+
+static SCM
+cwss_body (void *data)
+{
+  struct cwss_data *d = data;
+  return scm_internal_lazy_catch (d->tag, d->body, d->data, ss_handler, NULL);
+}
+
 SCM
-scm_ithrow (key, args, noreturn)
-     SCM key;
-     SCM args;
-     int noreturn;
-#endif
+scm_internal_stack_catch (SCM tag,
+                         scm_catch_body_t body,
+                         void *body_data,
+                         scm_catch_handler_t handler,
+                         void *handler_data)
 {
-  SCM jmpbuf;
-  SCM wind_goal;
+  struct cwss_data d;
+  d.tag = tag;
+  d.body = body;
+  d.data = body_data;
+  return scm_internal_catch (tag, cwss_body, &d, handler, handler_data);
+}
+
+
+\f
+/* body and handler functions for use with any of the above catch variants */
+
+/* This is a body function you can pass to scm_internal_catch if you
+   want the body to be like Scheme's `catch' --- a thunk.
+
+   BODY_DATA is a pointer to a scm_body_thunk_data structure, which
+   contains the Scheme procedure to invoke as the body, and the tag
+   we're catching.  */
+
+SCM
+scm_body_thunk (void *body_data)
+{
+  struct scm_body_thunk_data *c = (struct scm_body_thunk_data *) body_data;
+
+  return scm_apply (c->body_proc, SCM_EOL, SCM_EOL);
+}
+
+
+/* This is a handler function you can pass to scm_internal_catch if
+   you want the handler to act like Scheme's catch: (throw TAG ARGS ...)
+   applies a handler procedure to (TAG ARGS ...).
 
-  if (SCM_NIMP (key) && SCM_JMPBUFP (key))
+   If the user does a throw to this catch, this function runs a
+   handler procedure written in Scheme.  HANDLER_DATA is a pointer to
+   an SCM variable holding the Scheme procedure object to invoke.  It
+   ought to be a pointer to an automatic variable (i.e., one living on
+   the stack), or the procedure object should be otherwise protected
+   from GC.  */
+SCM
+scm_handle_by_proc (void *handler_data, SCM tag, SCM throw_args)
+{
+  SCM *handler_proc_p = (SCM *) handler_data;
+
+  return scm_apply (*handler_proc_p, scm_cons (tag, throw_args), SCM_EOL);
+}
+
+/* SCM_HANDLE_BY_PROC_CATCHING_ALL is like SCM_HANDLE_BY_PROC but
+   catches all throws that the handler might emit itself.  The handler
+   used for these `secondary' throws is SCM_HANDLE_BY_MESSAGE_NO_EXIT.  */
+
+struct hbpca_data {
+  SCM proc;
+  SCM args;
+};
+
+static SCM
+hbpca_body (void *body_data)
+{
+  struct hbpca_data *data = (struct hbpca_data *)body_data;
+  return scm_apply (data->proc, data->args, SCM_EOL);
+}
+
+SCM
+scm_handle_by_proc_catching_all (void *handler_data, SCM tag, SCM throw_args)
+{
+  SCM *handler_proc_p = (SCM *) handler_data;
+  struct hbpca_data data;
+  data.proc = *handler_proc_p;
+  data.args = scm_cons (tag, throw_args);
+
+  return scm_internal_catch (SCM_BOOL_T,
+                            hbpca_body, &data,
+                            scm_handle_by_message_noexit, NULL);
+}
+
+/* Derive the an exit status from the arguments to (quit ...).  */
+int
+scm_exit_status (SCM args)
+{
+  if (SCM_NNULLP (args))
+    {
+      SCM cqa = SCM_CAR (args);
+      
+      if (SCM_INUMP (cqa))
+       return (SCM_INUM (cqa));
+      else if (SCM_FALSEP (cqa))
+       return 1;
+    }
+  return 0;
+}
+       
+
+static void
+handler_message (void *handler_data, SCM tag, SCM args)
+{
+  char *prog_name = (char *) handler_data;
+  SCM p = scm_cur_errp;
+
+  if (scm_ilength (args) >= 3)
     {
-      jmpbuf = key;
-      if (noreturn)
+      SCM stack   = scm_make_stack (SCM_LIST1 (SCM_BOOL_T));
+      SCM subr    = SCM_CAR (args);
+      SCM message = SCM_CADR (args);
+      SCM parts   = SCM_CADDR (args);
+      SCM rest    = SCM_CDDDR (args);
+
+      if (SCM_BACKTRACE_P && SCM_NFALSEP (stack))
        {
-         SCM_ASSERT (JBACTIVE (jmpbuf), jmpbuf,
-                 "throw to dynamically inactive catch",
-                 s_throw);
+         scm_puts ("Backtrace:\n", p);
+         scm_display_backtrace (stack, p, SCM_UNDEFINED, SCM_UNDEFINED);
+         scm_newline (p);
        }
-      else if (!JBACTIVE (jmpbuf))
-       return SCM_UNSPECIFIED;
+      scm_display_error (stack, p, subr, message, parts, rest);
     }
   else
     {
-      SCM dynpair;
-      SCM hook;
+      if (! prog_name)
+       prog_name = "guile";
 
-      if (noreturn)
-       {
-         SCM_ASSERT (SCM_NIMP (key) && SCM_SYMBOLP (key), key, SCM_ARG1, s_throw);
-       }
-      else if (!(SCM_NIMP (key) && SCM_SYMBOLP (key)))
-       return SCM_UNSPECIFIED;
+      scm_puts (prog_name, p);
+      scm_puts (": ", p);
+
+      scm_puts ("uncaught throw to ", p);
+      scm_prin1 (tag, p, 0);
+      scm_puts (": ", p);
+      scm_prin1 (args, p, 1);
+      scm_putc ('\n', p);
+    }
+}
+
+
+/* This is a handler function to use if you want scheme to print a
+   message and die.  Useful for dealing with throws to uncaught keys
+   at the top level.
+
+   At boot time, we establish a catch-all that uses this as its handler.
+   1) If the user wants something different, they can use (catch #t
+   ...) to do what they like.
+   2) Outside the context of a read-eval-print loop, there isn't
+   anything else good to do; libguile should not assume the existence
+   of a read-eval-print loop.
+   3) Given that we shouldn't do anything complex, it's much more
+   robust to do it in C code.
+
+   HANDLER_DATA, if non-zero, is assumed to be a char * pointing to a
+   message header to print; if zero, we use "guile" instead.  That
+   text is followed by a colon, then the message described by ARGS.  */
+
+SCM
+scm_handle_by_message (void *handler_data, SCM tag, SCM args)
+{
+  if (SCM_NFALSEP (scm_eq_p (tag, SCM_CAR (scm_intern0 ("quit")))))
+    {
+      exit (scm_exit_status (args));
+    }
+
+  handler_message (handler_data, tag, args);
+  /* try to flush the error message first before the rest of the
+     ports: if any throw error, it currently causes a bus
+     exception.  */
+  exit (2);
+}
+
+
+/* This is just like scm_handle_by_message, but it doesn't exit; it
+   just returns #f.  It's useful in cases where you don't really know
+   enough about the body to handle things in a better way, but don't
+   want to let throws fall off the bottom of the wind list.  */
+SCM
+scm_handle_by_message_noexit (void *handler_data, SCM tag, SCM args)
+{
+  handler_message (handler_data, tag, args);
+
+  return SCM_BOOL_F;
+}
+
+
+SCM
+scm_handle_by_throw (void *handler_data, SCM tag, SCM args)
+{
+  scm_ithrow (tag, args, 1);
+  return SCM_UNSPECIFIED;  /* never returns */
+}
+
+
+\f
+/* the Scheme-visible CATCH and LAZY-CATCH functions */
+
+SCM_DEFINE (scm_catch, "catch", 3, 0, 0,
+           (SCM tag, SCM thunk, SCM handler),
+           "Invoke @var{thunk} in the dynamic context of @var{handler} for\n"
+           "exceptions matching @var{key}.  If thunk throws to the symbol @var{key},\n"
+           "then @var{handler} is invoked this way:\n\n"
+           "@example\n"
+           "(handler key args ...)\n"
+           "@end example\n\n"
+           "@var{key} is a symbol or #t.\n\n"
+           "@var{thunk} takes no arguments.  If @var{thunk} returns normally, that\n"
+           "is the return value of @code{catch}.\n\n"
+           "Handler is invoked outside the scope of its own @code{catch}.  If\n"
+           "@var{handler} again throws to the same key, a new handler from further\n"
+           "up the call chain is invoked.\n\n"
+           "If the key is @code{#t}, then a throw to @emph{any} symbol will match\n"
+           "this call to @code{catch}.")
+#define FUNC_NAME s_scm_catch
+{
+  struct scm_body_thunk_data c;
+
+  SCM_ASSERT (SCM_SYMBOLP(tag) || tag == SCM_BOOL_T,
+             tag, SCM_ARG1, FUNC_NAME);
+
+  c.tag = tag;
+  c.body_proc = thunk;
+
+  /* scm_internal_catch takes care of all the mechanics of setting up
+     a catch tag; we tell it to call scm_body_thunk to run the body,
+     and scm_handle_by_proc to deal with any throws to this catch.
+     The former receives a pointer to c, telling it how to behave.
+     The latter receives a pointer to HANDLER, so it knows who to call.  */
+  return scm_internal_catch (tag,
+                            scm_body_thunk, &c, 
+                            scm_handle_by_proc, &handler);
+}
+#undef FUNC_NAME
+
+
+SCM_DEFINE (scm_lazy_catch, "lazy-catch", 3, 0, 0,
+           (SCM tag, SCM thunk, SCM handler),
+           "")
+#define FUNC_NAME s_scm_lazy_catch
+{
+  struct scm_body_thunk_data c;
+
+  SCM_ASSERT (SCM_SYMBOLP(tag) || (tag == SCM_BOOL_T),
+             tag, SCM_ARG1, FUNC_NAME);
+
+  c.tag = tag;
+  c.body_proc = thunk;
+
+  /* scm_internal_lazy_catch takes care of all the mechanics of
+     setting up a lazy catch tag; we tell it to call scm_body_thunk to
+     run the body, and scm_handle_by_proc to deal with any throws to
+     this catch.  The former receives a pointer to c, telling it how
+     to behave.  The latter receives a pointer to HANDLER, so it knows
+     who to call.  */
+  return scm_internal_lazy_catch (tag,
+                                 scm_body_thunk, &c, 
+                                 scm_handle_by_proc, &handler);
+}
+#undef FUNC_NAME
+
+
+\f
+/* throwing */
 
-      dynpair = scm_sloppy_assq (key, scm_dynwinds);
+SCM_DEFINE (scm_throw, "throw", 1, 0, 1,
+           (SCM key, SCM args),
+           "Invoke the catch form matching @var{key}, passing @var{args} to the\n"
+           "@var{handler}.  \n\n"
+           "@var{key} is a symbol.  It will match catches of the same symbol or of\n"
+           "#t.\n\n"
+           "If there is no handler at all, an error is signaled.")
+#define FUNC_NAME s_scm_throw
+{
+  SCM_VALIDATE_SYMBOL (1,key);
+  /* May return if handled by lazy catch. */
+  return scm_ithrow (key, args, 1);
+}
+#undef FUNC_NAME
 
-      if (dynpair == SCM_BOOL_F)
-       dynpair = scm_sloppy_assq (SCM_BOOL_T, scm_dynwinds);
+SCM
+scm_ithrow (SCM key, SCM args, int noreturn)
+{
+  SCM jmpbuf = SCM_UNDEFINED;
+  SCM wind_goal;
 
-      hook = SCM_CDR (scm_bad_throw_vcell);
-      if ((dynpair == SCM_BOOL_F)
-         && (SCM_BOOL_T == scm_procedure_p (hook)))
+  SCM dynpair = SCM_UNDEFINED;
+  SCM winds;
+
+  /* Search the wind list for an appropriate catch.
+     "Waiter, please bring us the wind list." */
+  for (winds = scm_dynwinds; SCM_NIMP (winds); winds = SCM_CDR (winds))
+    {
+      if (! SCM_CONSP (winds))
+       abort ();
+
+      dynpair = SCM_CAR (winds);
+      if (SCM_CONSP (dynpair))
        {
-         SCM answer;
-         answer = scm_apply (hook, scm_cons (key, args), SCM_EOL);
+         SCM this_key = SCM_CAR (dynpair);
+
+         if (this_key == SCM_BOOL_T || this_key == key)
+           break;
        }
+    }
+
+  /* If we didn't find anything, abort.  scm_boot_guile should
+         have established a catch-all, but obviously things are
+         thoroughly screwed up.  */
+  if (winds == SCM_EOL)
+    abort ();
+
+      /* If the wind list is malformed, bail.  */
+  if (SCM_IMP (winds) || SCM_NCONSP (winds))
+    abort ();
       
-      if (dynpair != SCM_BOOL_F)
-       jmpbuf = SCM_CDR (dynpair);
+  if (dynpair != SCM_BOOL_F)
+    jmpbuf = SCM_CDR (dynpair);
+  else
+    {
+      if (!noreturn)
+       return SCM_UNSPECIFIED;
       else
        {
-         if (!noreturn)
-           return SCM_UNSPECIFIED;
-         else
-           {
-             scm_exitval = scm_cons (key, args);
-             scm_dowinds (SCM_EOL, scm_ilength (scm_dynwinds));
+         scm_exitval = scm_cons (key, args);
+         scm_dowinds (SCM_EOL, scm_ilength (scm_dynwinds));
 #ifdef DEBUG_EXTENSIONS
-             last_debug_info_frame = SCM_DFRAME (scm_rootcont);
+         scm_last_debug_frame = SCM_DFRAME (scm_rootcont);
 #endif
-             longjmp (SCM_JMPBUF (scm_rootcont), 1);
-           }
+         longjmp (SCM_JMPBUF (scm_rootcont), 1);
        }
     }
+
   for (wind_goal = scm_dynwinds;
        SCM_CDAR (wind_goal) != jmpbuf;
        wind_goal = SCM_CDR (wind_goal))
     ;
-  {
-    struct jmp_buf_and_retval * jbr;
-    jbr = (struct jmp_buf_and_retval *)JBJMPBUF (jmpbuf);
-    jbr->throw_tag = key;
-    jbr->retval = args;
-  }
-  scm_dowinds (wind_goal, scm_ilength (scm_dynwinds) - scm_ilength (wind_goal));
-#ifdef DEBUG_EXTENSIONS
-  last_debug_info_frame = JBSCM_DFRAME (jmpbuf);
-#endif
-  longjmp (*JBJMPBUF (jmpbuf), 1);
-}
 
+  /* Is a lazy catch?  In wind list entries for lazy catches, the key
+     is bound to a lazy_catch smob, not a jmpbuf.  */
+  if (SCM_LAZY_CATCH_P (jmpbuf))
+    {
+      struct lazy_catch *c = (struct lazy_catch *) SCM_CDR (jmpbuf);
+      SCM oldwinds = scm_dynwinds;
+      SCM handle, answer;
+      scm_dowinds (wind_goal, (scm_ilength (scm_dynwinds)
+                              - scm_ilength (wind_goal)));
+      SCM_REDEFER_INTS;
+      handle = scm_dynwinds;
+      scm_dynwinds = SCM_CDR (scm_dynwinds);
+      SCM_REALLOW_INTS;
+      answer = (c->handler) (c->handler_data, key, args);
+      SCM_REDEFER_INTS;
+      SCM_SETCDR (handle, scm_dynwinds);
+      scm_dynwinds = handle;
+      SCM_REALLOW_INTS;
+      scm_dowinds (oldwinds, (scm_ilength (scm_dynwinds)
+                             - scm_ilength (oldwinds)));
+      return answer;
+    }
 
-SCM_PROC(s_throw, "throw", 1, 0, 1, scm_throw);
-#ifdef __STDC__
-SCM
-scm_throw (SCM key, SCM args)
-#else
-SCM
-scm_throw (key, args)
-     SCM key;
-     SCM args;
+  /* Otherwise, it's a normal catch.  */
+  else if (SCM_JMPBUFP (jmpbuf))
+    {
+      struct jmp_buf_and_retval * jbr;
+      scm_dowinds (wind_goal, (scm_ilength (scm_dynwinds)
+                              - scm_ilength (wind_goal)));
+      jbr = (struct jmp_buf_and_retval *)JBJMPBUF (jmpbuf);
+      jbr->throw_tag = key;
+      jbr->retval = args;
+    }
+
+  /* Otherwise, it's some random piece of junk.  */
+  else
+    abort ();
+
+#ifdef DEBUG_EXTENSIONS
+  scm_last_debug_frame = SCM_JBDFRAME (jmpbuf);
 #endif
-{
-  scm_ithrow (key, args, 1);
-  return SCM_BOOL_F;  /* never really returns */
+  longjmp (*JBJMPBUF (jmpbuf), 1);
 }
 
 
-#ifdef __STDC__
-void
-scm_init_throw (void)
-#else
 void
 scm_init_throw ()
-#endif
 {
-  scm_tc16_jmpbuffer = scm_newsmob (&jbsmob);
-  scm_bad_throw_vcell = scm_sysintern ("%%bad-throw", SCM_BOOL_F);
+  scm_tc16_jmpbuffer = scm_make_smob_type_mfpe ("jmpbuffer",
+#ifdef DEBUG_EXTENSIONS
+                                               sizeof (scm_cell),
+                                               NULL, /* mark */
+                                               freejb,
+#else
+                                               0,
+                                               NULL, /* mark */
+                                               NULL,
+#endif
+                                               printjb,
+                                               NULL);
+
+  tc16_lazy_catch = scm_make_smob_type_mfpe ("lazy-catch", 0,
+                                            NULL,
+                                            NULL,
+                                            print_lazy_catch,
+                                            NULL);
 #include "throw.x"
 }
-