Improved error checking in bytevector->uint-list and bytevector->sint-list.
[bpt/guile.git] / libguile / gdbint.c
index bc0c73a..7a0ebc9 100644 (file)
@@ -1,60 +1,35 @@
 /* GDB interface for Guile
- * Copyright (C) 1996, 1997, 1999, 2000 Free Software Foundation
+ * Copyright (C) 1996,1997,1999,2000,2001,2002,2004,2009,2011,2012
+ * 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
- * 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 3 of
+ * the License, or (at your option) any later version.
  *
- * This program 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.
+ * 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 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 "libguile/_scm.h"
 
 #include <stdio.h>
+#include <string.h>
 #ifdef HAVE_UNISTD_H
 #include <unistd.h>
 #endif
 
-#include "libguile/tag.h"
 #include "libguile/strports.h"
 #include "libguile/read.h"
 #include "libguile/eval.h"
 \f
 
 /* Be carefull when this macro is true.
-   scm_gc_heap_lock is set during gc.
+   scm_gc_running_p is set during gc.
  */
-#define SCM_GC_P (scm_gc_heap_lock)
+#define SCM_GC_P (scm_gc_running_p)
 
 /* Macros that encapsulate blocks of code which can be called by the
  * debugger.
  */
 #define SCM_BEGIN_FOREIGN_BLOCK \
 do { \
-  old_ints = scm_ints_disabled; scm_ints_disabled = 1; \
-  old_gc = scm_block_gc; scm_block_gc = 1; \
   scm_print_carefully_p = 1; \
 } while (0)
 
@@ -101,13 +74,9 @@ do { \
 #define SCM_END_FOREIGN_BLOCK \
 do { \
   scm_print_carefully_p = 0; \
-  scm_block_gc = old_gc; \
-  scm_ints_disabled = old_ints; \
 } while (0)
 
 
-#define MSG_GUILE_NOT_INITIALIZED "*** Guile not initialized ***"
-
 #define RESET_STRING { gdb_output_length = 0; }
 
 #define SEND_STRING(str) \
@@ -133,56 +102,26 @@ int gdb_output_length;
 int scm_print_carefully_p;
 
 static SCM gdb_input_port;
-static int port_mark_p, stream_mark_p, string_mark_p;
-
-static SCM tok_buf;
-static int tok_buf_mark_p;
-
 static SCM gdb_output_port;
-static int old_ints, old_gc;
-
-
-static void
-unmark_port (SCM port)
-{
-  SCM stream, string;
-  port_mark_p = SCM_GC8MARKP (port);
-  SCM_CLRGC8MARK (port);
-  stream = SCM_PACK (SCM_STREAM (port));
-  stream_mark_p = SCM_GCMARKP (stream);
-  SCM_CLRGCMARK (stream);
-  string = SCM_CDR (stream);
-  string_mark_p = SCM_GC8MARKP (string);
-  SCM_CLRGC8MARK (string);
-}
-
-
-static void
-remark_port (SCM port)
-{
-  SCM stream = SCM_PACK (SCM_STREAM (port));
-  SCM string = SCM_CDR (stream);
-  if (string_mark_p) SCM_SETGC8MARK (string);
-  if (stream_mark_p) SCM_SETGCMARK (stream);
-  if (port_mark_p) SCM_SETGC8MARK (port);
-}
 
 
 int
 gdb_maybe_valid_type_p (SCM value)
 {
-  return SCM_IMP (value) || scm_cellp (value);
+  return SCM_IMP (value); /*  || scm_in_heap_p (value); */ /* FIXME: What to
+                                                             do? */
 }
 
 
 int
 gdb_read (char *str)
 {
+#if 0
   SCM ans;
   int status = 0;
   RESET_STRING;
   /* Need to be restrictive about what to read? */
-  if (SCM_GC_P)
+  if (1)  /* (SCM_GC_P) */ /* FIXME */
     {
       char *p;
       for (p = str; *p != '\0'; ++p)
@@ -210,14 +149,13 @@ gdb_read (char *str)
     }
   SCM_BEGIN_FOREIGN_BLOCK;
   unmark_port (gdb_input_port);
-  scm_seek (gdb_input_port, SCM_INUM0, SCM_MAKINUM (SEEK_SET));
+  scm_seek (gdb_input_port, SCM_INUM0, scm_from_int (SEEK_SET));
   scm_puts (str, gdb_input_port);
   scm_truncate_file (gdb_input_port, SCM_UNDEFINED);
-  scm_seek (gdb_input_port, SCM_INUM0, SCM_MAKINUM (SEEK_SET));
+  scm_seek (gdb_input_port, SCM_INUM0, scm_from_int (SEEK_SET));
+
   /* Read one object */
-  tok_buf_mark_p = SCM_GC8MARKP (tok_buf);
-  SCM_CLRGC8MARK (tok_buf);
-  ans = scm_lreadr (&tok_buf, gdb_input_port, &ans);
+  ans = scm_read (gdb_input_port);
   if (SCM_GC_P)
     {
       if (SCM_NIMP (ans))
@@ -228,15 +166,16 @@ gdb_read (char *str)
        }
     }
   gdb_result = ans;
-  /* Protect answer from future GC */
+  /* Protect answer from future GC (FIXME: still needed with BDW-GC?) */
   if (SCM_NIMP (ans))
     scm_permanent_object (ans);
 exit:
-  if (tok_buf_mark_p)
-    SCM_SETGC8MARK (tok_buf);
   remark_port (gdb_input_port);
   SCM_END_FOREIGN_BLOCK;
   return status;
+#else
+  abort ();
+#endif
 }
 
 
@@ -244,11 +183,6 @@ int
 gdb_eval (SCM exp)
 {
   RESET_STRING;
-  if (SCM_IMP (exp))
-    {
-      gdb_result = exp;
-      return 0;
-    }
   if (SCM_GC_P)
     {
       SEND_STRING ("Can't evaluate lisp expressions during gc");
@@ -256,8 +190,7 @@ gdb_eval (SCM exp)
     }
   SCM_BEGIN_FOREIGN_BLOCK;
   {
-    SCM env = scm_top_level_env (SCM_TOP_LEVEL_LOOKUP_CLOSURE);
-    gdb_result = scm_permanent_object (scm_ceval (exp, env));
+    gdb_result = scm_permanent_object (scm_primitive_eval (exp));
   }
   SCM_END_FOREIGN_BLOCK;
   return 0;
@@ -274,11 +207,11 @@ gdb_print (SCM obj)
       RESET_STRING;
       SCM_BEGIN_FOREIGN_BLOCK;
       /* Reset stream */
-      scm_seek (gdb_output_port, SCM_INUM0, SCM_MAKINUM (SEEK_SET));
+      scm_seek (gdb_output_port, SCM_INUM0, scm_from_int (SEEK_SET));
       scm_write (obj, gdb_output_port);
       scm_truncate_file (gdb_output_port, SCM_UNDEFINED);
       {
-       scm_port *pt = SCM_PTAB_ENTRY (gdb_output_port);
+       scm_t_port *pt = SCM_PTAB_ENTRY (gdb_output_port);
 
        scm_flush (gdb_output_port);
        *(pt->write_buf + pt->read_buf_size) = 0;
@@ -301,10 +234,7 @@ gdb_binding (SCM name, SCM value)
     }
   SCM_BEGIN_FOREIGN_BLOCK;
   {
-    SCM vcell = scm_sym2vcell (name,
-                              SCM_TOP_LEVEL_LOOKUP_CLOSURE,
-                              SCM_BOOL_T);
-    SCM_SETCDR (vcell, value);
+    scm_define (name, value);
   }
   SCM_END_FOREIGN_BLOCK;
   return 0;
@@ -317,20 +247,16 @@ scm_init_gdbint ()
   SCM port;
 
   scm_print_carefully_p = 0;
-  
-  port = scm_mkstrport (SCM_INUM0,
-                       scm_make_string (SCM_MAKINUM (0), SCM_UNDEFINED),
+
+  port = scm_mkstrport (SCM_INUM0, SCM_BOOL_F,
                        SCM_OPN | SCM_WRTNG,
                        s);
   gdb_output_port = scm_permanent_object (port);
-  
-  port = scm_mkstrport (SCM_INUM0,
-                       scm_make_string (SCM_MAKINUM (0), SCM_UNDEFINED),
+
+  port = scm_mkstrport (SCM_INUM0, SCM_BOOL_F,
                        SCM_OPN | SCM_RDNG | SCM_WRTNG,
                        s);
   gdb_input_port = scm_permanent_object (port);
-
-  tok_buf = scm_permanent_object (scm_makstr (30L, 0));
 }
 
 /*