Merge commit 'e20d7001c3f7150400169fecb0bf0eefdf122fe2' into vm-check
[bpt/guile.git] / libguile / backtrace.c
index 7cd5632..a8afcdf 100644 (file)
 /* Printing of backtraces and error messages
- * Copyright (C) 1996, 1997, 1998, 1999 Free Software Foundation
+ * Copyright (C) 1996,1997,1998,1999,2000,2001, 2003, 2004, 2006 Free Software Foundation
  *
- * 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
- * the Free Software Foundation; either version 2, or (at your option)
- * any later version.
+ * 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.
  *
- * This program is distributed in the hope that it will be useful,
+ * 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 General Public License for more details.
+ * 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 General Public License
- * along with this software; see the file COPYING.  If not, write to
- * 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.
- *
- * The exception is that, if you link the GUILE library with other files
- * to produce an executable, this does not by itself cause the
- * resulting executable to be covered by the GNU General Public License.
- * Your use of that executable is in no way restricted on account of
- * linking the GUILE library code into it.
- *
- * This exception does not however invalidate any other reasons why
- * the executable file might be covered by the GNU General Public License.
- *
- * This exception applies only to the code released by the
- * Free Software Foundation under the name GUILE.  If you copy
- * code from other Free Software Foundation releases into a copy of
- * GUILE, as the General Public License permits, the exception does
- * not apply to the code that you add in this way.  To avoid misleading
- * anyone as to the status of such modified files, you must delete
- * this exception notice from them.
- *
- * 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.
- *
- * The author can be reached at djurfeldt@nada.kth.se
- * Mikael Djurfeldt, SANS/NADA KTH, 10044 STOCKHOLM, SWEDEN */
-
-/* Software engineering face-lift by Greg J. Badros, 11-Dec-1999,
-   gjb@cs.washington.edu, http://www.cs.washington.edu/homes/gjb */
+ * 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
 
+#ifdef HAVE_CONFIG_H
+#  include <config.h>
+#endif
 
 #include <stdio.h>
 #include <ctype.h>
+#include <assert.h>
 
-#include "_scm.h"
+#include "libguile/_scm.h"
 
 #ifdef HAVE_UNISTD_H
 #include <unistd.h>
 #endif
+#ifdef HAVE_IO_H
+#include <io.h>
+#endif
 
-#include "stacks.h"
-#include "srcprop.h"
-#include "genio.h"
-#include "struct.h"
-#include "strports.h"
-#include "throw.h"
-#include "fluids.h"
-
-#include "validate.h"
-#include "backtrace.h"
+#include "libguile/stacks.h"
+#include "libguile/srcprop.h"
+#include "libguile/struct.h"
+#include "libguile/strports.h"
+#include "libguile/throw.h"
+#include "libguile/fluids.h"
+#include "libguile/ports.h"
+#include "libguile/strings.h"
+#include "libguile/dynwind.h"
+
+#include "libguile/validate.h"
+#include "libguile/lang.h"
+#include "libguile/backtrace.h"
+#include "libguile/filesys.h"
+#include "libguile/private-options.h"
 
 /* {Error reporting and backtraces}
- * (A first approximation.)
  *
  * Note that these functions shouldn't generate errors themselves.
  */
 
-#ifndef SCM_RECKLESS
+/* Print parameters for error messages. */
+
+#define DISPLAY_ERROR_MESSAGE_MAX_LEVEL   7
+#define DISPLAY_ERROR_MESSAGE_MAX_LENGTH 10
+
+/* Print parameters for failing expressions in error messages.
+ * (See also `print_params' below for backtrace print parameters.)
+ */
+
+#define DISPLAY_EXPRESSION_MAX_LEVEL      2
+#define DISPLAY_EXPRESSION_MAX_LENGTH     3
+
 #undef SCM_ASSERT
 #define SCM_ASSERT(_cond, _arg, _pos, _subr) \
        if (!(_cond)) \
           return SCM_BOOL_F;
-#endif
 
-SCM scm_the_last_stack_fluid;
+SCM scm_the_last_stack_fluid_var;
 
 static void
 display_header (SCM source, SCM port)
 {
-  SCM fname = (SCM_MEMOIZEDP (source)
-              ? scm_source_property (source, scm_sym_filename)
-              : SCM_BOOL_F);
-  if (SCM_STRINGP (fname))
+  if (SCM_MEMOIZEDP (source))
     {
-      scm_prin1 (fname, port, 0);
-      scm_putc (':', port);
-      scm_intprint (SCM_INUM (scm_source_property (source, scm_sym_line)) + 1,
-                   10,
-                   port);
-      scm_putc (':', port);
-      scm_intprint (SCM_INUM (scm_source_property (source, scm_sym_column)) + 1,
-                   10,
-                   port);
+      SCM fname = scm_source_property (source, scm_sym_filename);
+      SCM line = scm_source_property (source, scm_sym_line);
+      SCM col = scm_source_property (source, scm_sym_column);
+
+      /* Dirk:FIXME:: Maybe we should store the _port_ rather than the
+       * filename with the source properties?  Then we could in case of
+       * non-file ports give at least some more details than just
+       * "<unnamed port>". */
+      if (scm_is_true (fname))
+       scm_prin1 (fname, port, 0);
+      else
+       scm_puts ("<unnamed port>", port);
+
+      if (scm_is_true (line) && scm_is_true (col))
+       {
+         scm_putc (':', port);
+         scm_intprint (scm_to_long (line) + 1, 10, port);
+         scm_putc (':', port);
+         scm_intprint (scm_to_long (col) + 1, 10, port);
+       }
     }
   else
     scm_puts ("ERROR", port);
@@ -106,31 +105,80 @@ display_header (SCM source, SCM port)
 }
 
 
+struct display_error_message_data {
+  SCM message;
+  SCM args;
+  SCM port;
+  scm_print_state *pstate;
+  int old_fancyp;
+  int old_level;
+  int old_length;
+};
+
+static SCM
+display_error_message (struct display_error_message_data *d)
+{
+  if (scm_is_string (d->message) && scm_is_true (scm_list_p (d->args)))
+    scm_simple_format (d->port, d->message, d->args);
+  else
+    scm_display (d->message, d->port);
+  scm_newline (d->port);
+  return SCM_UNSPECIFIED;
+}
+
+static void
+before_display_error_message (struct display_error_message_data *d)
+{
+  scm_print_state *pstate = d->pstate;
+  d->old_fancyp = pstate->fancyp;
+  d->old_level  = pstate->level;
+  d->old_length = pstate->length;
+  pstate->fancyp = 1;
+  pstate->level  = DISPLAY_ERROR_MESSAGE_MAX_LEVEL;
+  pstate->length = DISPLAY_ERROR_MESSAGE_MAX_LENGTH;
+}
+
+static void
+after_display_error_message (struct display_error_message_data *d)
+{
+  scm_print_state *pstate = d->pstate;
+  pstate->fancyp = d->old_fancyp;
+  pstate->level  = d->old_level;
+  pstate->length = d->old_length;
+}
+
 void
 scm_display_error_message (SCM message, SCM args, SCM port)
 {
-  if (SCM_ROSTRINGP (message) && SCM_NFALSEP (scm_list_p (args)))
-    {
-      scm_simple_format (port, message, args);
-      scm_newline (port);
-    }
-  else
-    {
-      scm_prin1 (message, port, 0);
-      scm_putc ('\n', port);
-    }
+  struct display_error_message_data d;
+  SCM print_state;
+  scm_print_state *pstate;
+
+  port = scm_i_port_with_print_state (port, SCM_UNDEFINED);
+  print_state = SCM_PORT_WITH_PS_PS (port);
+  pstate = SCM_PRINT_STATE (print_state);
+  
+  d.message = message;
+  d.args = args;
+  d.port = port;
+  d.pstate = pstate;
+  scm_internal_dynamic_wind ((scm_t_guard) before_display_error_message,
+                            (scm_t_inner) display_error_message,
+                            (scm_t_guard) after_display_error_message,
+                            &d,
+                            &d);
 }
 
 static void
-display_expression (SCM frame,SCM pname,SCM source,SCM port)
+display_expression (SCM frame, SCM pname, SCM source, SCM port)
 {
   SCM print_state = scm_make_print_state ();
   scm_print_state *pstate = SCM_PRINT_STATE (print_state);
   pstate->writingp = 0;
   pstate->fancyp = 1;
-  pstate->level = 2;
-  pstate->length = 3;
-  if (SCM_ROSTRINGP (pname))
+  pstate->level  = DISPLAY_EXPRESSION_MAX_LEVEL;
+  pstate->length = DISPLAY_EXPRESSION_MAX_LENGTH;
+  if (scm_is_symbol (pname) || scm_is_string (pname))
     {
       if (SCM_FRAMEP (frame)
          && SCM_FRAME_EVAL_ARGS_P (frame))
@@ -142,14 +190,14 @@ display_expression (SCM frame,SCM pname,SCM source,SCM port)
        {
          scm_puts (" in expression ", port);
          pstate->writingp = 1;
-         scm_iprin1 (scm_unmemoize (source), port, pstate);
+         scm_iprin1 (scm_i_unmemoize_expr (source), port, pstate);
        }
     }
-  else if (SCM_NIMP (source))
+  else if (SCM_MEMOIZEDP (source))
     {
       scm_puts ("In expression ", port);
       pstate->writingp = 1;
-      scm_iprin1 (scm_unmemoize (source), port, pstate);
+      scm_iprin1 (scm_i_unmemoize_expr (source), port, pstate);
     }
   scm_puts (":\n", port);
   scm_free_print_state (print_state);
@@ -169,26 +217,25 @@ display_error_body (struct display_error_args *a)
 {
   SCM current_frame = SCM_BOOL_F;
   SCM source = SCM_BOOL_F;
-  SCM pname = SCM_BOOL_F;
   SCM prev_frame = SCM_BOOL_F;
+  SCM pname = a->subr;
 
-  if (SCM_DEBUGGINGP
+  if (scm_debug_mode_p
       && SCM_STACKP (a->stack)
       && SCM_STACK_LENGTH (a->stack) > 0)
     {
       current_frame = scm_stack_ref (a->stack, SCM_INUM0);
       source = SCM_FRAME_SOURCE (current_frame);
       prev_frame = SCM_FRAME_PREV (current_frame);
-      if (!SCM_MEMOIZEDP (source)
-         && prev_frame != SCM_BOOL_F)
+      if (!SCM_MEMOIZEDP (source) && scm_is_true (prev_frame))
        source = SCM_FRAME_SOURCE (prev_frame);
-      if (SCM_FRAME_PROC_P (current_frame)
-         && scm_procedure_p (SCM_FRAME_PROC (current_frame)) == SCM_BOOL_T)
+      if (!scm_is_symbol (pname)
+         && !scm_is_string (pname)
+         && SCM_FRAME_PROC_P (current_frame)
+         && scm_is_true (scm_procedure_p (SCM_FRAME_PROC (current_frame))))
        pname = scm_procedure_name (SCM_FRAME_PROC (current_frame));
     }
-  if (!SCM_ROSTRINGP (pname))
-    pname = a->subr;
-  if (SCM_ROSTRINGP (pname) || SCM_MEMOIZEDP (source))
+  if (scm_is_symbol (pname) || scm_is_string (pname) || SCM_MEMOIZEDP (source))
     {
       display_header (source, a->port);
       display_expression (current_frame, pname, source, a->port);
@@ -210,7 +257,7 @@ struct display_error_handler_data {
    try to print all objects, we would enter an infinite loop. */
 static SCM
 display_error_handler (struct display_error_handler_data *data,
-                      SCM tag, SCM args)
+                      SCM tag, SCM args SCM_UNUSED)
 {
   SCM print_state = scm_make_print_state ();
   scm_puts ("\nException during displaying of ", data->port);
@@ -221,10 +268,14 @@ display_error_handler (struct display_error_handler_data *data,
   return SCM_UNSPECIFIED;
 }
 
-SCM_DEFINE (scm_display_error, "display-error", 6, 0, 0,
-           (SCM stack, SCM port, SCM subr, SCM message, SCM args, SCM rest),
-"")
-#define FUNC_NAME s_scm_display_error
+
+/* The function scm_i_display_error prints out a detailed error message.  This
+ * function will be called directly within libguile to signal error messages.
+ * No parameter checks will be performed by scm_i_display_error.  Thus, User
+ * code should rather use the function scm_display_error.
+ */
+void
+scm_i_display_error (SCM stack, SCM port, SCM subr, SCM message, SCM args, SCM rest)
 {
   struct display_error_args a;
   struct display_error_handler_data data;
@@ -237,12 +288,31 @@ SCM_DEFINE (scm_display_error, "display-error", 6, 0, 0,
   data.mode = "error";
   data.port = port;
   scm_internal_catch (SCM_BOOL_T,
-                     (scm_catch_body_t) display_error_body, &a,
-                     (scm_catch_handler_t) display_error_handler, &data);
+                     (scm_t_catch_body) display_error_body, &a,
+                     (scm_t_catch_handler) display_error_handler, &data);
+}
+
+
+SCM_DEFINE (scm_display_error, "display-error", 6, 0, 0,
+           (SCM stack, SCM port, SCM subr, SCM message, SCM args, SCM rest),
+           "Display an error message to the output port @var{port}.\n"
+           "@var{stack} is the saved stack for the error, @var{subr} is\n"
+           "the name of the procedure in which the error occurred and\n"
+           "@var{message} is the actual error message, which may contain\n"
+           "formatting instructions. These will format the arguments in\n"
+           "the list @var{args} accordingly.  @var{rest} is currently\n"
+           "ignored.")
+#define FUNC_NAME s_scm_display_error
+{
+  SCM_VALIDATE_OUTPUT_PORT (2, port);
+
+  scm_i_display_error (stack, port, subr, message, args, rest);
+
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 
+
 typedef struct {
   int level;
   int length;
@@ -260,7 +330,9 @@ static print_params_t *print_params = default_print_params;
 #ifdef GUILE_DEBUG
 SCM_DEFINE (scm_set_print_params_x, "set-print-params!", 1, 0, 0,
            (SCM params),
-"")
+           "Set the print parameters to the values from @var{params}.\n"
+           "@var{params} must be a list of two-element lists which must\n"
+           "hold two integer values.")
 #define FUNC_NAME s_scm_set_print_params_x
 {
   int i;
@@ -269,24 +341,21 @@ SCM_DEFINE (scm_set_print_params_x, "set-print-params!", 1, 0, 0,
   print_params_t *new_params;
 
   SCM_VALIDATE_NONEMPTYLIST_COPYLEN (2, params, n);
-  for (ls = params; SCM_NNULLP (ls); ls = SCM_CDR (ls))
+  for (ls = params; !SCM_NULL_OR_NIL_P (ls); ls = SCM_CDR (ls))
     SCM_ASSERT (scm_ilength (SCM_CAR (params)) == 2
-               && SCM_INUMP (SCM_CAAR (ls))
-               && SCM_INUM (SCM_CAAR (ls)) >= 0
-               && SCM_INUMP (SCM_CADAR (ls))
-               && SCM_INUM (SCM_CADAR (ls)) >= 0,
+               && scm_is_unsigned_integer (SCM_CAAR (ls), 0, INT_MAX)
+               && scm_is_unsigned_integer (SCM_CADAR (ls), 0, INT_MAX),
                params,
                SCM_ARG2,
                s_scm_set_print_params_x);
-  new_params = scm_must_malloc (n * sizeof (print_params_t),
-                               FUNC_NAME);
+  new_params = scm_malloc (n * sizeof (print_params_t));
   if (print_params != default_print_params)
-    scm_must_free (print_params);
+    free (print_params);
   print_params = new_params;
   for (i = 0; i < n; ++i)
     {
-      print_params[i].level = SCM_INUM (SCM_CAAR (params));
-      print_params[i].length = SCM_INUM (SCM_CADAR (params));
+      print_params[i].level = scm_to_int (SCM_CAAR (params));
+      print_params[i].length = scm_to_int (SCM_CADAR (params));
       params = SCM_CDR (params);
     }
   n_print_params = n;
@@ -304,16 +373,15 @@ indent (int n, SCM port)
 }
 
 static void
-display_frame_expr (char *hdr,SCM exp,char *tlr,int indentation,SCM sport,SCM port,scm_print_state *pstate)
+display_frame_expr (char *hdr, SCM exp, char *tlr, int indentation, SCM sport, SCM port, scm_print_state *pstate)
 {
-  SCM string;
   int i = 0, n;
-  scm_ptob_descriptor *ptob = scm_ptobs + SCM_PTOBNUM (sport);
+  scm_t_ptob_descriptor *ptob = scm_ptobs + SCM_PTOBNUM (sport);
   do
     {
       pstate->length = print_params[i].length;
       ptob->seek (sport, 0, SEEK_SET);
-      if (SCM_CONSP (exp))
+      if (scm_is_pair (exp))
        {
          pstate->level = print_params[i].level - 1;
          scm_iprlist (hdr, exp, tlr[0], sport, pstate);
@@ -330,30 +398,19 @@ display_frame_expr (char *hdr,SCM exp,char *tlr,int indentation,SCM sport,SCM po
     }
   while (indentation + n > SCM_BACKTRACE_WIDTH && i < n_print_params);
   ptob->truncate (sport, n);
-  string = scm_strport_to_string (sport);
-  /* Remove control characters */
-  for (i = 0; i < n; ++i)
-    if (iscntrl (SCM_CHARS (string)[i]))
-      SCM_CHARS (string)[i] = ' ';
-  /* Truncate */
-  if (indentation + n > SCM_BACKTRACE_WIDTH)
-    {
-      n = SCM_BACKTRACE_WIDTH - indentation;
-      SCM_CHARS (string)[n - 1] = '$';
-    }
       
-  scm_lfwrite (SCM_CHARS (string), n, port);
+  scm_display (scm_strport_to_string (sport), port);
 }
 
 static void
-display_application (SCM frame,int indentation,SCM sport,SCM port,scm_print_state *pstate)
+display_application (SCM frame, int indentation, SCM sport, SCM port, scm_print_state *pstate)
 {
   SCM proc = SCM_FRAME_PROC (frame);
-  SCM name = (SCM_NFALSEP (scm_procedure_p (proc))
+  SCM name = (scm_is_true (scm_procedure_p (proc))
              ? scm_procedure_name (proc)
              : SCM_BOOL_F);
   display_frame_expr ("[",
-                     scm_cons (SCM_NFALSEP (name) ? name : proc,
+                     scm_cons (scm_is_true (name) ? name : proc,
                                SCM_FRAME_ARGS (frame)),
                      SCM_FRAME_EVAL_ARGS_P (frame) ? " ..." : "]",
                      indentation,
@@ -364,18 +421,18 @@ display_application (SCM frame,int indentation,SCM sport,SCM port,scm_print_stat
 
 SCM_DEFINE (scm_display_application, "display-application", 1, 2, 0, 
            (SCM frame, SCM port, SCM indent),
-"")
+           "Display a procedure application @var{frame} to the output port\n"
+           "@var{port}. @var{indent} specifies the indentation of the\n"
+           "output.")
 #define FUNC_NAME s_scm_display_application
 {
-  SCM_VALIDATE_FRAME (1,frame);
+  SCM_VALIDATE_FRAME (1, frame);
   if (SCM_UNBNDP (port))
-    port = scm_cur_outp;
+    port = scm_current_output_port ();
   else
-    SCM_VALIDATE_OPOUTPORT (2,port);
+    SCM_VALIDATE_OPOUTPORT (2, port);
   if (SCM_UNBNDP (indent))
     indent = SCM_INUM0;
-  else
-    SCM_VALIDATE_INUM (3,indent);
   
   if (SCM_FRAME_PROC_P (frame))
     /* Display an application. */
@@ -385,7 +442,7 @@ SCM_DEFINE (scm_display_application, "display-application", 1, 2, 0,
       
       /* Create a string port used for adaptation of printing parameters. */
       sport = scm_mkstrport (SCM_INUM0,
-                            scm_make_string (SCM_MAKINUM (240),
+                            scm_make_string (scm_from_int (240),
                                              SCM_UNDEFINED),
                             SCM_OPN | SCM_WRTNG,
                             FUNC_NAME);
@@ -396,7 +453,7 @@ SCM_DEFINE (scm_display_application, "display-application", 1, 2, 0,
       pstate->writingp = 1;
       pstate->fancyp = 1;
       
-      display_application (frame, SCM_INUM (indent), sport, port, pstate);
+      display_application (frame, scm_to_int (indent), sport, port, pstate);
       return SCM_BOOL_T;
     }
   else
@@ -404,8 +461,108 @@ SCM_DEFINE (scm_display_application, "display-application", 1, 2, 0,
 }
 #undef FUNC_NAME
 
+SCM_SYMBOL (sym_base, "base");
+
+static void
+display_backtrace_get_file_line (SCM frame, SCM *file, SCM *line)
+{
+  SCM source = SCM_FRAME_SOURCE (frame);
+  *file = *line = SCM_BOOL_F;
+  if (SCM_MEMOIZEDP (source))
+    {
+      *file = scm_source_property (source, scm_sym_filename);
+      *line = scm_source_property (source, scm_sym_line);
+    }
+  else if (scm_is_pair (source)
+           && scm_is_pair (scm_cdr (source))
+           && scm_is_pair (scm_cddr (source))
+           && !scm_is_pair (scm_cdddr (source)))
+    {
+      /* (addr . (filename . (line . column))), from vm compilation */
+      *file = scm_cadr (source);
+      *line = scm_caddr (source);
+    }
+}
+
+static void
+display_backtrace_file (frame, last_file, port, pstate)
+     SCM frame;
+     SCM *last_file;
+     SCM port;
+     scm_print_state *pstate;
+{
+  SCM file, line;
+
+  display_backtrace_get_file_line (frame, &file, &line);
+
+  if (scm_is_eq (file, *last_file))
+    return;
+
+  *last_file = file;
+
+  scm_puts ("In ", port);
+  if (scm_is_false (file))
+    if (scm_is_false (line))
+      scm_puts ("unknown file", port);
+    else
+      scm_puts ("current input", port);
+  else
+    {
+      pstate->writingp = 0;
+      scm_iprin1 (file, port, pstate);
+      pstate->writingp = 1;
+    }
+  scm_puts (":\n", port);
+}
+
+static void
+display_backtrace_file_and_line (SCM frame, SCM port, scm_print_state *pstate)
+{
+  SCM file, line;
+
+  display_backtrace_get_file_line (frame, &file, &line);
+
+  if (scm_is_eq (SCM_PACK (SCM_SHOW_FILE_NAME), sym_base))
+    {
+      if (scm_is_false (file))
+       {
+         if (scm_is_false (line))
+           scm_putc ('?', port);
+         else
+           scm_puts ("<stdin>", port);
+       }
+      else
+       {
+         pstate -> writingp = 0;
+#ifdef HAVE_POSIX
+         scm_iprin1 ((scm_is_string (file)?
+                      scm_basename (file, SCM_UNDEFINED) : file),
+                     port, pstate);
+#else
+         scm_iprin1 (file, port, pstate);
+#endif
+         pstate -> writingp = 1;
+       }
+
+      scm_putc (':', port);
+    }
+  else if (scm_is_true (line))
+    {
+      int i, j=0;
+      for (i = scm_to_int (line)+1; i > 0; i = i/10, j++)
+       ;
+      indent (4-j, port);
+    }
+
+  if (scm_is_false (line))
+    scm_puts ("   ?", port);
+  else
+    scm_intprint (scm_to_int (line) + 1, 10, port);
+  scm_puts (": ", port);
+}
+
 static void
-display_frame (SCM frame,int nfield,int indentation,SCM sport,SCM port,scm_print_state *pstate)
+display_frame (SCM frame, int nfield, int indentation, SCM sport, SCM port, scm_print_state *pstate)
 {
   int n, i, j;
 
@@ -416,6 +573,10 @@ display_frame (SCM frame,int nfield,int indentation,SCM sport,SCM port,scm_print
       scm_puts ("...\n", port);
     }
 
+  /* display file name and line number */
+  if (scm_is_true (SCM_PACK (SCM_SHOW_FILE_NAME)))
+    display_backtrace_file_and_line (frame, port, pstate);
+
   /* Check size of frame number. */
   n = SCM_FRAME_NUMBER (frame);
   for (i = 0, j = n; j > 0; ++i) j /= 10;
@@ -424,7 +585,7 @@ display_frame (SCM frame,int nfield,int indentation,SCM sport,SCM port,scm_print
   indent (nfield - (i ? i : 1), port);
 
   /* Frame number. */
-  scm_iprin1 (SCM_MAKINUM (n), port, pstate);
+  scm_iprin1 (scm_from_int (n), port, pstate);
 
   /* Real frame marker */
   scm_putc (SCM_FRAME_REAL_P (frame) ? '*' : ' ', port);
@@ -439,14 +600,14 @@ display_frame (SCM frame,int nfield,int indentation,SCM sport,SCM port,scm_print
     /* Display a special form. */
     {
       SCM source = SCM_FRAME_SOURCE (frame);
-      SCM copy = (SCM_CONSP (source) 
+      SCM copy = (scm_is_pair (source) 
                  ? scm_source_property (source, scm_sym_copy)
                  : SCM_BOOL_F);
       SCM umcopy = (SCM_MEMOIZEDP (source)
-                   ? scm_unmemoize (source)
+                   ? scm_i_unmemoize_expr (source)
                    : SCM_BOOL_F);
       display_frame_expr ("(",
-                         SCM_CONSP (copy) ? copy : umcopy,
+                         scm_is_pair (copy) ? copy : umcopy,
                          ")",
                          nfield + 1 + indentation,
                          sport,
@@ -468,33 +629,29 @@ struct display_backtrace_args {
   SCM port;
   SCM first;
   SCM depth;
+  SCM highlight_objects;
 };
 
 static SCM
-display_backtrace_body(struct display_backtrace_args *a)
+display_backtrace_body (struct display_backtrace_args *a)
 #define FUNC_NAME "display_backtrace_body"
 {
   int n_frames, beg, end, n, i, j;
   int nfield, indent_p, indentation;
   SCM frame, sport, print_state;
+  SCM last_file;
   scm_print_state *pstate;
 
   a->port = SCM_COERCE_OUTPORT (a->port);
 
   /* Argument checking and extraction. */
-  SCM_ASSERT (SCM_STACKP (a->stack),
-             a->stack,
-             SCM_ARG1,
-             s_display_backtrace);
-  SCM_ASSERT (SCM_OPOUTPORTP (a->port),
-             a->port,
-             SCM_ARG2,
-             s_display_backtrace);
-  n_frames = SCM_INUM (scm_stack_length (a->stack));
-  n = SCM_INUMP (a->depth) ? SCM_INUM (a->depth) : SCM_BACKTRACE_DEPTH;
+  SCM_VALIDATE_STACK (1, a->stack);
+  SCM_VALIDATE_OPOUTPORT (2, a->port);
+  n_frames = scm_to_int (scm_stack_length (a->stack));
+  n = scm_is_integer (a->depth) ? scm_to_int (a->depth) : SCM_BACKTRACE_DEPTH;
   if (SCM_BACKWARDS_P)
     {
-      beg = SCM_INUMP (a->first) ? SCM_INUM (a->first) : 0;
+      beg = scm_is_integer (a->first) ? scm_to_int (a->first) : 0;
       end = beg + n - 1;
       if (end >= n_frames)
        end = n_frames - 1;
@@ -502,9 +659,9 @@ display_backtrace_body(struct display_backtrace_args *a)
     }
   else
     {
-      if (SCM_INUMP (a->first))
+      if (scm_is_integer (a->first))
        {
-         beg = SCM_INUM (a->first);
+         beg = scm_to_int (a->first);
          end = beg - n + 1;
          if (end < 0)
            end = 0;
@@ -523,7 +680,7 @@ display_backtrace_body(struct display_backtrace_args *a)
 
   /* Create a string port used for adaptation of printing parameters. */
   sport = scm_mkstrport (SCM_INUM0,
-                        scm_make_string (SCM_MAKINUM (240), SCM_UNDEFINED),
+                        scm_make_string (scm_from_int (240), SCM_UNDEFINED),
                         SCM_OPN | SCM_WRTNG,
                         FUNC_NAME);
 
@@ -532,14 +689,17 @@ display_backtrace_body(struct display_backtrace_args *a)
   pstate = SCM_PRINT_STATE (print_state);
   pstate->writingp = 1;
   pstate->fancyp = 1;
+  pstate->highlight_objects = a->highlight_objects;
 
   /* First find out if it's reasonable to do indentation. */
   if (SCM_BACKWARDS_P)
     indent_p = 0;
   else
     {
+      unsigned int j;
+
       indent_p = 1;
-      frame = scm_stack_ref (a->stack, SCM_MAKINUM (beg));
+      frame = scm_stack_ref (a->stack, scm_from_int (beg));
       for (i = 0, j = 0; i < n; ++i)
        {
          if (SCM_FRAME_REAL_P (frame))
@@ -556,30 +716,43 @@ display_backtrace_body(struct display_backtrace_args *a)
     }
   
   /* Determine size of frame number field. */
-  j = SCM_FRAME_NUMBER (scm_stack_ref (a->stack, SCM_MAKINUM (end)));
+  j = SCM_FRAME_NUMBER (scm_stack_ref (a->stack, scm_from_int (end)));
   for (i = 0; j > 0; ++i) j /= 10;
   nfield = i ? i : 1;
   
   /* Print frames. */
-  frame = scm_stack_ref (a->stack, SCM_MAKINUM (beg));
+  frame = scm_stack_ref (a->stack, scm_from_int (beg));
   indentation = 1;
-  display_frame (frame, nfield, indentation, sport, a->port, pstate);
-  for (i = 1; i < n; ++i)
+  last_file = SCM_UNDEFINED;
+  for (i = 0; i < n; ++i)
     {
+      if (!scm_is_eq (SCM_PACK (SCM_SHOW_FILE_NAME), sym_base))
+       display_backtrace_file (frame, &last_file, a->port, pstate);
+
+      display_frame (frame, nfield, indentation, sport, a->port, pstate);
       if (indent_p && SCM_FRAME_EVAL_ARGS_P (frame))
        ++indentation;
-      frame = SCM_BACKWARDS_P ? SCM_FRAME_PREV (frame) : SCM_FRAME_NEXT (frame);
-      display_frame (frame, nfield, indentation, sport, a->port, pstate);
+      frame = (SCM_BACKWARDS_P ? 
+              SCM_FRAME_PREV (frame) : SCM_FRAME_NEXT (frame));
     }
 
+  scm_remember_upto_here_1 (print_state);
+  
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 
-SCM_DEFINE (scm_display_backtrace, "display-backtrace", 2, 2, 0, 
-           (SCM stack, SCM port, SCM first, SCM depth),
-"")
-#define FUNC_NAME s_scm_display_backtrace
+SCM_DEFINE (scm_display_backtrace_with_highlights, "display-backtrace", 2, 3, 0, 
+           (SCM stack, SCM port, SCM first, SCM depth, SCM highlights),
+           "Display a backtrace to the output port @var{port}.  @var{stack}\n"
+           "is the stack to take the backtrace from, @var{first} specifies\n"
+           "where in the stack to start and @var{depth} how many frames\n"
+           "to display.  @var{first} and @var{depth} can be @code{#f},\n"
+           "which means that default values will be used.\n"
+           "If @var{highlights} is given it should be a list; the elements\n"
+           "of this list will be highlighted wherever they appear in the\n"
+           "backtrace.")
+#define FUNC_NAME s_scm_display_backtrace_with_highlights
 {
   struct display_backtrace_args a;
   struct display_error_handler_data data;
@@ -587,57 +760,90 @@ SCM_DEFINE (scm_display_backtrace, "display-backtrace", 2, 2, 0,
   a.port  = port;
   a.first = first;
   a.depth = depth;
+  if (SCM_UNBNDP (highlights))
+    a.highlight_objects = SCM_EOL;
+  else
+    a.highlight_objects = highlights;
   data.mode = "backtrace";
   data.port = port;
   scm_internal_catch (SCM_BOOL_T,
-                     (scm_catch_body_t) display_backtrace_body, &a,
-                     (scm_catch_handler_t) display_error_handler, &data);
+                     (scm_t_catch_body) display_backtrace_body, &a,
+                     (scm_t_catch_handler) display_error_handler, &data);
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 
-SCM_VCELL (scm_has_shown_backtrace_hint_p_var, "has-shown-backtrace-hint?");
+SCM
+scm_display_backtrace (SCM stack, SCM port, SCM first, SCM depth)
+{
+  return scm_display_backtrace_with_highlights (stack, port, first, depth,
+                                               SCM_EOL);
+}
 
-SCM_DEFINE (scm_backtrace, "backtrace", 0, 0, 0, 
-           (),
-"")
-#define FUNC_NAME s_scm_backtrace
+SCM_VARIABLE (scm_has_shown_backtrace_hint_p_var, "has-shown-backtrace-hint?");
+
+SCM_DEFINE (scm_backtrace_with_highlights, "backtrace", 0, 1, 0, 
+           (SCM highlights),
+           "Display a backtrace of the stack saved by the last error\n"
+           "to the current output port.  If @var{highlights} is given\n"
+           "it should be a list; the elements of this list will be\n"
+           "highlighted wherever they appear in the backtrace.")
+#define FUNC_NAME s_scm_backtrace_with_highlights
 {
-  SCM the_last_stack = scm_fluid_ref (SCM_CDR (scm_the_last_stack_fluid));
-  if (SCM_NFALSEP (the_last_stack))
+  SCM port = scm_current_output_port ();
+  SCM the_last_stack =
+    scm_fluid_ref (SCM_VARIABLE_REF (scm_the_last_stack_fluid_var));
+
+  if (SCM_UNBNDP (highlights))
+    highlights = SCM_EOL;
+
+  if (scm_is_true (the_last_stack))
     {
-      scm_newline (scm_cur_outp);
-      scm_puts ("Backtrace:\n", scm_cur_outp);
-      scm_display_backtrace (the_last_stack,
-                            scm_cur_outp,
-                            SCM_UNDEFINED,
-                            SCM_UNDEFINED);
-      scm_newline (scm_cur_outp);
-      if (SCM_FALSEP (SCM_CDR (scm_has_shown_backtrace_hint_p_var))
+      scm_newline (port);
+      scm_puts ("Backtrace:\n", port);
+      scm_display_backtrace_with_highlights (the_last_stack,
+                                            port,
+                                            SCM_BOOL_F,
+                                            SCM_BOOL_F,
+                                            highlights);
+      scm_newline (port);
+      if (scm_is_false (SCM_VARIABLE_REF (scm_has_shown_backtrace_hint_p_var))
          && !SCM_BACKTRACE_P)
        {
          scm_puts ("Type \"(debug-enable 'backtrace)\" if you would like "
                    "a backtrace\n"
                    "automatically if an error occurs in the future.\n",
-                   scm_cur_outp);
-         SCM_SETCDR (scm_has_shown_backtrace_hint_p_var, SCM_BOOL_T);
+                   port);
+         SCM_VARIABLE_SET (scm_has_shown_backtrace_hint_p_var, SCM_BOOL_T);
        }
     }
   else
     {
-      scm_puts ("No backtrace available.\n", scm_cur_outp);
+      scm_puts ("No backtrace available.\n", port);
     }
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 
+SCM
+scm_backtrace (void)
+{
+  return scm_backtrace_with_highlights (SCM_EOL);
+}
+
 \f
 
 void
 scm_init_backtrace ()
 {
   SCM f = scm_make_fluid ();
-  scm_the_last_stack_fluid = scm_sysintern ("the-last-stack", f);
+  scm_the_last_stack_fluid_var = scm_c_define ("the-last-stack", f);
 
-#include "backtrace.x"
+#include "libguile/backtrace.x"
 }
+
+/*
+  Local Variables:
+  c-file-style: "gnu"
+  End:
+*/