Merge branch 'master' into boehm-demers-weiser-gc
[bpt/guile.git] / libguile / environments.c
index ced455b..78ccd28 100644 (file)
@@ -1,51 +1,30 @@
-/* Copyright (C) 1999, 2000 Free Software Foundation, Inc.
+/* Copyright (C) 1999,2000,2001, 2003, 2006, 2008 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 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.  */
+ * 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
+ */
 
 \f
+#ifdef HAVE_CONFIG_H
+# include <config.h>
+#endif
 
 #include "libguile/_scm.h"
 #include "libguile/alist.h"
 #include "libguile/eval.h"
-#include "libguile/gh.h"
 #include "libguile/hash.h"
+#include "libguile/list.h"
 #include "libguile/ports.h"
 #include "libguile/smob.h"
 #include "libguile/symbols.h"
 
 \f
 
-long scm_tc16_environment;
-long scm_tc16_observer;
+scm_t_bits scm_tc16_environment;
+scm_t_bits scm_tc16_observer;
+#define DEFAULT_OBARRAY_SIZE 31
+
+SCM scm_system_environment;
 
 \f
 
@@ -116,50 +98,46 @@ scm_error_environment_immutable_location (const char *func, SCM env, SCM symbol)
 SCM
 scm_make_environment (void *type)
 {
-  SCM env;
-
-  SCM_NEWCELL (env);
-  SCM_SET_CELL_WORD_1 (env, type);
-  SCM_SET_CELL_TYPE (env, scm_tc16_environment);
-
-  return env;
+  SCM_RETURN_NEWSMOB (scm_tc16_environment, type);
 }
 
 
 SCM_DEFINE (scm_environment_p, "environment?", 1, 0, 0, 
            (SCM obj),
-           "Return #t if OBJ is an environment, or #f otherwise.")
+           "Return @code{#t} if @var{obj} is an environment, or @code{#f}\n"
+           "otherwise.")
 #define FUNC_NAME s_scm_environment_p
 {
-  return SCM_BOOL (SCM_ENVIRONMENT_P (obj));
+  return scm_from_bool (SCM_ENVIRONMENT_P (obj));
 }
 #undef FUNC_NAME
 
 
 SCM_DEFINE (scm_environment_bound_p, "environment-bound?", 2, 0, 0, 
            (SCM env, SCM sym),
-           "Return #t if SYM is bound in ENV, or #f otherwise.")
+           "Return @code{#t} if @var{sym} is bound in @var{env}, or\n"
+           "@code{#f} otherwise.")
 #define FUNC_NAME s_scm_environment_bound_p
 {
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, FUNC_NAME);
 
-  return SCM_BOOL (SCM_ENVIRONMENT_BOUND_P (env, sym));
+  return scm_from_bool (SCM_ENVIRONMENT_BOUND_P (env, sym));
 }
 #undef FUNC_NAME
 
 
 SCM_DEFINE (scm_environment_ref, "environment-ref", 2, 0, 0,
            (SCM env, SCM sym),
-           "Return the value of the location bound to SYM in ENV.\n"
-           "If SYM is unbound in ENV, signal an environment:unbound\n"
-           "error.")
+           "Return the value of the location bound to @var{sym} in\n"
+           "@var{env}. If @var{sym} is unbound in @var{env}, signal an\n"
+           "@code{environment:unbound} error.")
 #define FUNC_NAME s_scm_environment_ref
 {
   SCM val;
 
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, FUNC_NAME);
 
   val = SCM_ENVIRONMENT_REF (env, sym);
 
@@ -179,7 +157,7 @@ SCM
 scm_c_environment_ref (SCM env, SCM sym)
 {
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, "scm_c_environment_ref");
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, "scm_c_environment_ref");
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, "scm_c_environment_ref");
   return SCM_ENVIRONMENT_REF (env, sym);
 }
 
@@ -187,40 +165,47 @@ scm_c_environment_ref (SCM env, SCM sym)
 static SCM
 environment_default_folder (SCM proc, SCM symbol, SCM value, SCM tail)
 {
-  return gh_call3 (proc, symbol, value, tail);
+  return scm_call_3 (proc, symbol, value, tail);
 }
 
 
 SCM_DEFINE (scm_environment_fold, "environment-fold", 3, 0, 0, 
            (SCM env, SCM proc, SCM init),
-           "Iterate over all the bindings in ENV, accumulating some value.\n"
-           "For each binding in ENV, apply PROC to the symbol bound, its\n"
-           "value, and the result from the previous application of PROC.\n"
-           "Use INIT as PROC's third argument the first time PROC is\n"
-           "applied.\n"
-           "If ENV contains no bindings, this function simply returns INIT.\n"
-           "If ENV binds the symbol sym1 to the value val1, sym2 to val2,\n"
-           "and so on, then this procedure computes:\n"
+           "Iterate over all the bindings in @var{env}, accumulating some\n"
+           "value.\n"
+           "For each binding in @var{env}, apply @var{proc} to the symbol\n"
+           "bound, its value, and the result from the previous application\n"
+           "of @var{proc}.\n"
+           "Use @var{init} as @var{proc}'s third argument the first time\n"
+           "@var{proc} is applied.\n"
+           "If @var{env} contains no bindings, this function simply returns\n"
+           "@var{init}.\n"
+           "If @var{env} binds the symbol sym1 to the value val1, sym2 to\n"
+           "val2, and so on, then this procedure computes:\n"
+           "@lisp\n"
            "  (proc sym1 val1\n"
            "        (proc sym2 val2\n"
            "              ...\n"
            "              (proc symn valn\n"
            "                    init)))\n"
-           "Each binding in ENV will be processed exactly once.\n"
-           "environment-fold makes no guarantees about the order in which\n"
-           "the bindings are processed.\n"
+           "@end lisp\n"
+           "Each binding in @var{env} will be processed exactly once.\n"
+           "@code{environment-fold} makes no guarantees about the order in\n"
+           "which the bindings are processed.\n"
            "Here is a function which, given an environment, constructs an\n"
            "association list representing that environment's bindings,\n"
            "using environment-fold:\n"
+           "@lisp\n"
            "  (define (environment->alist env)\n"
            "    (environment-fold env\n"
            "                      (lambda (sym val tail)\n"
            "                        (cons (cons sym val) tail))\n"
-           "                      '()))")
+           "                      '()))\n"
+           "@end lisp")
 #define FUNC_NAME s_scm_environment_fold
 {
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_EQ_P (scm_procedure_p (proc), SCM_BOOL_T), 
+  SCM_ASSERT (scm_is_true (scm_procedure_p (proc)), 
              proc, SCM_ARG2, FUNC_NAME);
 
   return SCM_ENVIRONMENT_FOLD (env, environment_default_folder, proc, init);
@@ -245,24 +230,25 @@ scm_c_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM init
 
 SCM_DEFINE (scm_environment_define, "environment-define", 3, 0, 0, 
            (SCM env, SCM sym, SCM val),
-           "Bind SYM to a new location containing VAL in ENV. If SYM is\n"
-           "already bound to another location in ENV and the binding is\n"
-           "mutable, that binding is replaced.  The new binding and\n"
-           "location are both mutable. The return value is unspecified.\n"
-           "If SYM is already bound in ENV, and the binding is immutable,\n"
-           "signal an environment:immutable-binding error.")
+           "Bind @var{sym} to a new location containing @var{val} in\n"
+           "@var{env}. If @var{sym} is already bound to another location\n"
+           "in @var{env} and the binding is mutable, that binding is\n"
+           "replaced.  The new binding and location are both mutable. The\n"
+           "return value is unspecified.\n"
+           "If @var{sym} is already bound in @var{env}, and the binding is\n"
+           "immutable, signal an @code{environment:immutable-binding} error.")
 #define FUNC_NAME s_scm_environment_define
 {
   SCM status;
 
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, FUNC_NAME);
 
   status = SCM_ENVIRONMENT_DEFINE (env, sym, val);
 
-  if (SCM_EQ_P (status, SCM_ENVIRONMENT_SUCCESS))
+  if (scm_is_eq (status, SCM_ENVIRONMENT_SUCCESS))
     return SCM_UNSPECIFIED;
-  else if (SCM_EQ_P (status, SCM_ENVIRONMENT_BINDING_IMMUTABLE))
+  else if (scm_is_eq (status, SCM_ENVIRONMENT_BINDING_IMMUTABLE))
     scm_error_environment_immutable_binding (FUNC_NAME, env, sym);
   else
     abort();
@@ -272,22 +258,23 @@ SCM_DEFINE (scm_environment_define, "environment-define", 3, 0, 0,
 
 SCM_DEFINE (scm_environment_undefine, "environment-undefine", 2, 0, 0, 
            (SCM env, SCM sym),
-           "Remove any binding for SYM from ENV. If SYM is unbound in ENV,\n"
-           "do nothing.  The return value is unspecified.\n"
-           "If SYM is already bound in ENV, and the binding is immutable,\n"
-           "signal an environment:immutable-binding error.")
+           "Remove any binding for @var{sym} from @var{env}. If @var{sym}\n"
+           "is unbound in @var{env}, do nothing.  The return value is\n"
+           "unspecified.\n"
+           "If @var{sym} is already bound in @var{env}, and the binding is\n"
+           "immutable, signal an @code{environment:immutable-binding} error.")
 #define FUNC_NAME s_scm_environment_undefine
 {
   SCM status;
 
   SCM_ASSERT(SCM_ENVIRONMENT_P(env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT(SCM_SYMBOLP(sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT(scm_is_symbol(sym), sym, SCM_ARG2, FUNC_NAME);
 
   status = SCM_ENVIRONMENT_UNDEFINE (env, sym);
 
-  if (SCM_EQ_P (status, SCM_ENVIRONMENT_SUCCESS))
+  if (scm_is_eq (status, SCM_ENVIRONMENT_SUCCESS))
     return SCM_UNSPECIFIED;
-  else if (SCM_EQ_P (status, SCM_ENVIRONMENT_BINDING_IMMUTABLE))
+  else if (scm_is_eq (status, SCM_ENVIRONMENT_BINDING_IMMUTABLE))
     scm_error_environment_immutable_binding (FUNC_NAME, env, sym);
   else
     abort();
@@ -297,25 +284,27 @@ SCM_DEFINE (scm_environment_undefine, "environment-undefine", 2, 0, 0,
 
 SCM_DEFINE (scm_environment_set_x, "environment-set!", 3, 0, 0, 
            (SCM env, SCM sym, SCM val),
-           "If ENV binds SYM to some location, change that location's\n"
-           "value to VAL.  The return value is unspecified.\n"
-           "If SYM is not bound in ENV, signal an environment:unbound\n"
-           "error.  If ENV binds SYM to an immutable location, signal an\n"
-           "environment:immutable-location error.")
+           "If @var{env} binds @var{sym} to some location, change that\n"
+           "location's value to @var{val}.  The return value is\n"
+           "unspecified.\n"
+           "If @var{sym} is not bound in @var{env}, signal an\n"
+           "@code{environment:unbound} error.  If @var{env} binds @var{sym}\n"
+           "to an immutable location, signal an\n"
+           "@code{environment:immutable-location} error.")
 #define FUNC_NAME s_scm_environment_set_x
 {
   SCM status;
 
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, FUNC_NAME);
 
   status = SCM_ENVIRONMENT_SET (env, sym, val);
 
-  if (SCM_EQ_P (status, SCM_ENVIRONMENT_SUCCESS))
+  if (scm_is_eq (status, SCM_ENVIRONMENT_SUCCESS))
     return SCM_UNSPECIFIED;
   else if (SCM_UNBNDP (status))
     scm_error_environment_unbound (FUNC_NAME, env, sym);
-  else if (SCM_EQ_P (status, SCM_ENVIRONMENT_LOCATION_IMMUTABLE))
+  else if (scm_is_eq (status, SCM_ENVIRONMENT_LOCATION_IMMUTABLE))
     scm_error_environment_immutable_binding (FUNC_NAME, env, sym);
   else
     abort();
@@ -325,30 +314,32 @@ SCM_DEFINE (scm_environment_set_x, "environment-set!", 3, 0, 0,
 
 SCM_DEFINE (scm_environment_cell, "environment-cell", 3, 0, 0, 
            (SCM env, SCM sym, SCM for_write),
-           "Return the value cell which ENV binds to SYM, or #f if the\n"
-           "binding does not live in a value cell.\n"
-           "The argument FOR-WRITE indicates whether the caller intends\n"
-           "to modify the variable's value by mutating the value cell.  If\n"
-           "the variable is immutable, then environment-cell signals an\n"
-           "environment:immutable-location error.\n"
-           "If SYM is unbound in ENV, signal an environment:unbound error.\n"
+           "Return the value cell which @var{env} binds to @var{sym}, or\n"
+           "@code{#f} if the binding does not live in a value cell.\n"
+           "The argument @var{for-write} indicates whether the caller\n"
+           "intends to modify the variable's value by mutating the value\n"
+           "cell.  If the variable is immutable, then\n"
+           "@code{environment-cell} signals an\n"
+           "@code{environment:immutable-location} error.\n"
+           "If @var{sym} is unbound in @var{env}, signal an\n"
+           "@code{environment:unbound} error.\n"
            "If you use this function, you should consider using\n"
-           "environment-observe, to be notified when SYM gets re-bound to\n"
-           "a new value cell, or becomes undefined.")
+           "@code{environment-observe}, to be notified when @var{sym} gets\n"
+           "re-bound to a new value cell, or becomes undefined.")
 #define FUNC_NAME s_scm_environment_cell
 {
   SCM location;
 
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, FUNC_NAME);
-  SCM_ASSERT (SCM_BOOLP (for_write), for_write, SCM_ARG3, FUNC_NAME);
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, FUNC_NAME);
+  SCM_ASSERT (scm_is_bool (for_write), for_write, SCM_ARG3, FUNC_NAME);
 
-  location = SCM_ENVIRONMENT_CELL (env, sym, !SCM_FALSEP (for_write));
+  location = SCM_ENVIRONMENT_CELL (env, sym, scm_is_true (for_write));
   if (!SCM_IMP (location))
     return location;
   else if (SCM_UNBNDP (location))
     scm_error_environment_unbound (FUNC_NAME, env, sym);
-  else if (SCM_EQ_P (location, SCM_ENVIRONMENT_LOCATION_IMMUTABLE))
+  else if (scm_is_eq (location, SCM_ENVIRONMENT_LOCATION_IMMUTABLE))
     scm_error_environment_immutable_location (FUNC_NAME, env, sym);
   else /* no cell */
     return location;
@@ -366,7 +357,7 @@ SCM
 scm_c_environment_cell(SCM env, SCM sym, int for_write)
 {
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, "scm_c_environment_cell");
-  SCM_ASSERT (SCM_SYMBOLP (sym), sym, SCM_ARG2, "scm_c_environment_cell");
+  SCM_ASSERT (scm_is_symbol (sym), sym, SCM_ARG2, "scm_c_environment_cell");
 
   return SCM_ENVIRONMENT_CELL (env, sym, for_write);
 }
@@ -375,17 +366,18 @@ scm_c_environment_cell(SCM env, SCM sym, int for_write)
 static void
 environment_default_observer (SCM env, SCM proc)
 {
-  gh_call1 (proc, env);
+  scm_call_1 (proc, env);
 }
 
 
 SCM_DEFINE (scm_environment_observe, "environment-observe", 2, 0, 0, 
            (SCM env, SCM proc),
-           "Whenever ENV's bindings change, apply PROC to ENV.\n"
+           "Whenever @var{env}'s bindings change, apply @var{proc} to\n"
+           "@var{env}.\n"
            "This function returns an object, token, which you can pass to\n"
-           "environment-unobserve to remove PROC from the set of\n"
-           "procedures observing ENV.  The type and value of token is\n"
-           "unspecified.")
+           "@code{environment-unobserve} to remove @var{proc} from the set\n"
+           "of procedures observing @var{env}.  The type and value of\n"
+           "token is unspecified.")
 #define FUNC_NAME s_scm_environment_observe
 {
   SCM_ASSERT (SCM_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -398,9 +390,10 @@ SCM_DEFINE (scm_environment_observe, "environment-observe", 2, 0, 0,
 SCM_DEFINE (scm_environment_observe_weak, "environment-observe-weak", 2, 0, 0,
            (SCM env, SCM proc),
            "This function is the same as environment-observe, except that\n"
-           "the reference ENV retains to PROC is a weak reference. This\n"
-           "means that, if there are no other live, non-weak references\n"
-           "to PROC, it will be garbage-collected, and dropped from ENV's\n"
+           "the reference @var{env} retains to @var{proc} is a weak\n"
+           "reference. This means that, if there are no other live,\n"
+           "non-weak references to @var{proc}, it will be\n"
+           "garbage-collected, and dropped from @var{env}'s\n"
            "list of observing procedures.")
 #define FUNC_NAME s_scm_environment_observe_weak
 {
@@ -433,10 +426,11 @@ scm_c_environment_observe (SCM env, scm_environment_observer proc, SCM data, int
 SCM_DEFINE (scm_environment_unobserve, "environment-unobserve", 1, 0, 0, 
            (SCM token),
            "Cancel the observation request which returned the value\n"
-           "TOKEN.  The return value is unspecified.\n"
-           "If a call (environment-observe env proc) returns token, then\n"
-           "the call (environment-unobserve token) will cause proc to no\n"
-           "longer be called when env's bindings change.")
+           "@var{token}.  The return value is unspecified.\n"
+           "If a call @code{(environment-observe env proc)} returns\n"
+           "@var{token}, then the call @code{(environment-unobserve token)}\n"
+           "will cause @var{proc} to no longer be called when @var{env}'s\n"
+           "bindings change.")
 #define FUNC_NAME s_scm_environment_unobserve
 {
   SCM env;
@@ -451,22 +445,9 @@ SCM_DEFINE (scm_environment_unobserve, "environment-unobserve", 1, 0, 0,
 #undef FUNC_NAME
 
 
-static SCM
-mark_environment (SCM env)
-{
-  return (*(SCM_ENVIRONMENT_FUNCS (env)->mark)) (env);
-}
-
-
-static scm_sizet
-free_environment (SCM env)
-{
-  return (*(SCM_ENVIRONMENT_FUNCS (env)->free)) (env);
-}
-
 
 static int
-print_environment (SCM env, SCM port, scm_print_state *pstate)
+environment_print (SCM env, SCM port, scm_print_state *pstate)
 {
   return (*(SCM_ENVIRONMENT_FUNCS (env)->print)) (env, port, pstate);
 }
@@ -475,30 +456,15 @@ print_environment (SCM env, SCM port, scm_print_state *pstate)
 
 /* observers */
 
-static SCM
-mark_observer (SCM observer)
-{
-  scm_gc_mark (SCM_OBSERVER_ENVIRONMENT (observer));
-  scm_gc_mark (SCM_OBSERVER_DATA (observer));
-  return SCM_BOOL_F;
-}
-
-
-static scm_sizet
-free_observer (SCM observer_smob)
-{
-  return 0;
-}
-
 
 static int
-print_observer (SCM type, SCM port, scm_print_state *pstate)
+observer_print (SCM type, SCM port, scm_print_state *pstate SCM_UNUSED)
 {
-  SCM address = scm_ulong2num (SCM_UNPACK (type));
-  SCM base16 = scm_number_to_string (address, SCM_MAKINUM (16));
+  SCM address = scm_from_size_t (SCM_UNPACK (type));
+  SCM base16 = scm_number_to_string (address, scm_from_int (16));
 
   scm_puts ("#<observer ", port);
-  scm_puts (SCM_STRING_CHARS (base16), port);
+  scm_display (base16, port);
   scm_puts (">", port);
 
   return 1;
@@ -521,10 +487,13 @@ print_observer (SCM type, SCM port, scm_print_state *pstate)
 static SCM
 obarray_enter (SCM obarray, SCM symbol, SCM data)
 {
-  scm_sizet hash = SCM_SYMBOL_HASH (symbol) % SCM_VECTOR_LENGTH (obarray);
+  size_t hash = scm_i_symbol_hash (symbol) % SCM_HASHTABLE_N_BUCKETS (obarray);
   SCM entry = scm_cons (symbol, data);
-  SCM slot = scm_cons (entry, SCM_VELTS (obarray)[hash]);
-  SCM_VELTS (obarray)[hash] = slot;
+  SCM slot = scm_cons (entry, SCM_HASHTABLE_BUCKET (obarray, hash));
+  SCM_SET_HASHTABLE_BUCKET  (obarray, hash, slot);
+  SCM_HASHTABLE_INCREMENT (obarray);
+  if (SCM_HASHTABLE_N_ITEMS (obarray) > SCM_HASHTABLE_UPPER (obarray))
+    scm_i_rehash (obarray, scm_i_hash_symbol, 0, "obarray_enter");
 
   return entry;
 }
@@ -537,23 +506,28 @@ obarray_enter (SCM obarray, SCM symbol, SCM data)
 static SCM
 obarray_replace (SCM obarray, SCM symbol, SCM data)
 {
-  scm_sizet hash = SCM_SYMBOL_HASH (symbol) % SCM_VECTOR_LENGTH (obarray);
+  size_t hash = scm_i_symbol_hash (symbol) % SCM_HASHTABLE_N_BUCKETS (obarray);
   SCM new_entry = scm_cons (symbol, data);
   SCM lsym;
   SCM slot;
 
-  for (lsym = SCM_VELTS (obarray)[hash]; !SCM_NULLP (lsym); lsym = SCM_CDR (lsym))
+  for (lsym = SCM_HASHTABLE_BUCKET (obarray, hash);
+       !scm_is_null (lsym);
+       lsym = SCM_CDR (lsym))
     {
       SCM old_entry = SCM_CAR (lsym);
-      if (SCM_EQ_P (SCM_CAR (old_entry), symbol))
+      if (scm_is_eq (SCM_CAR (old_entry), symbol))
        {
          SCM_SETCAR (lsym, new_entry);
          return old_entry;
        }
     }
 
-  slot = scm_cons (new_entry, SCM_VELTS (obarray)[hash]);
-  SCM_VELTS (obarray)[hash] = slot;
+  slot = scm_cons (new_entry, SCM_HASHTABLE_BUCKET (obarray, hash));
+  SCM_SET_HASHTABLE_BUCKET (obarray, hash, slot);
+  SCM_HASHTABLE_INCREMENT (obarray);
+  if (SCM_HASHTABLE_N_ITEMS (obarray) > SCM_HASHTABLE_UPPER (obarray))
+    scm_i_rehash (obarray, scm_i_hash_symbol, 0, "obarray_replace");
 
   return SCM_BOOL_F;
 }
@@ -565,13 +539,15 @@ obarray_replace (SCM obarray, SCM symbol, SCM data)
 static SCM
 obarray_retrieve (SCM obarray, SCM sym)
 {
-  scm_sizet hash = SCM_SYMBOL_HASH (sym) % SCM_VECTOR_LENGTH (obarray);
+  size_t hash = scm_i_symbol_hash (sym) % SCM_HASHTABLE_N_BUCKETS (obarray);
   SCM lsym;
 
-  for (lsym = SCM_VELTS (obarray)[hash]; !SCM_NULLP (lsym); lsym = SCM_CDR (lsym))
+  for (lsym = SCM_HASHTABLE_BUCKET (obarray, hash);
+       !scm_is_null (lsym);
+       lsym = SCM_CDR (lsym))
     {
       SCM entry = SCM_CAR (lsym);
-      if (SCM_EQ_P (SCM_CAR (entry), sym))
+      if (scm_is_eq (SCM_CAR (entry), sym))
        return entry;
     }
 
@@ -586,36 +562,32 @@ obarray_retrieve (SCM obarray, SCM sym)
 static SCM
 obarray_remove (SCM obarray, SCM sym)
 {
-  scm_sizet hash = SCM_SYMBOL_HASH (sym) % SCM_VECTOR_LENGTH (obarray);
-  SCM lsym;
-  SCM *lsymp;
+  size_t hash = scm_i_symbol_hash (sym) % SCM_HASHTABLE_N_BUCKETS (obarray);
+  SCM table_entry = SCM_HASHTABLE_BUCKET (obarray, hash);
+  SCM handle = scm_sloppy_assq (sym, table_entry);
 
-  /* Dirk:FIXME:: gc problem due to use of &SCM_VELTS[hash] */
-  for (lsym = *(lsymp = &SCM_VELTS (obarray)[hash]);
-       !SCM_NULLP (lsym);
-       lsym = *(lsymp = SCM_CDRLOC (lsym)))
+  if (scm_is_pair (handle))
     {
-      SCM entry = SCM_CAR (lsym);
-      if (SCM_EQ_P (SCM_CAR (entry), sym))
-       {
-         *lsymp = SCM_CDR (lsym);
-         return entry;
-       }
+      SCM new_table_entry = scm_delq1_x (handle, table_entry);
+      SCM_SET_HASHTABLE_BUCKET (obarray, hash, new_table_entry);
+      SCM_HASHTABLE_DECREMENT (obarray);
     }
-  return SCM_BOOL_F;
+
+  return handle;
 }
 
 
 static void
 obarray_remove_all (SCM obarray)
 {
-  scm_sizet size = SCM_VECTOR_LENGTH (obarray);
-  scm_sizet i;
+  size_t size = SCM_HASHTABLE_N_BUCKETS (obarray);
+  size_t i;
 
   for (i = 0; i < size; i++)
     {
-      SCM_VELTS (obarray)[i] = SCM_EOL;
+      SCM_SET_HASHTABLE_BUCKET (obarray, i, SCM_EOL);
     }
+  SCM_SET_HASHTABLE_N_ITEMS (obarray, 0);
 }
 
 \f
@@ -644,9 +616,9 @@ struct core_environments_base {
 #define CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR(env) \
   (CORE_ENVIRONMENTS_BASE (env)->weak_observers)
 #define CORE_ENVIRONMENT_WEAK_OBSERVERS(env) \
-  (SCM_VELTS (CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR (env)) [0])
+  (scm_c_vector_ref (CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR (env), 0))
 #define SCM_SET_CORE_ENVIRONMENT_WEAK_OBSERVERS(env, v) \
-  (SCM_VELTS (CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR (env)) [0] = (v))
+  (scm_c_vector_set_x (CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR (env), 0, (v)))
 
 \f
 
@@ -655,11 +627,10 @@ core_environments_observe (SCM env, scm_environment_observer proc, SCM data, int
 {
   SCM observer;
 
-  SCM_NEWCELL2 (observer);
-  SCM_SET_CELL_OBJECT_1 (observer, env);
-  SCM_SET_CELL_OBJECT_2 (observer, data);
-  SCM_SET_CELL_WORD_3 (observer, proc);
-  SCM_SET_CELL_TYPE (observer, scm_tc16_observer);
+  SCM_NEWSMOB3 (observer, scm_tc16_observer,
+               SCM_UNPACK (env),
+               SCM_UNPACK (data),
+               (scm_t_bits) proc);
 
   if (!weak_p)
     {
@@ -688,32 +659,33 @@ core_environments_unobserve (SCM env, SCM observer)
        ? CORE_ENVIRONMENT_WEAK_OBSERVERS (env)
        : CORE_ENVIRONMENT_OBSERVERS (env);
 
-      if (!SCM_NULLP (l))
+      if (!scm_is_null (l))
        {
          SCM rest = SCM_CDR (l);
          SCM first = handling_weaks
            ? SCM_CDAR (l)
            : SCM_CAR (l);
 
-         if (SCM_EQ_P (first, observer))
+         if (scm_is_eq (first, observer))
            {
              /* Remove the first observer */
-             handling_weaks
-               ? SCM_SET_CORE_ENVIRONMENT_WEAK_OBSERVERS (env, rest)
-               : SCM_SET_CORE_ENVIRONMENT_OBSERVERS (env, rest);
+             if (handling_weaks)
+               SCM_SET_CORE_ENVIRONMENT_WEAK_OBSERVERS (env, rest);
+              else
+                SCM_SET_CORE_ENVIRONMENT_OBSERVERS (env, rest);
              return;
            }
 
          do {
            SCM rest = SCM_CDR (l);
 
-           if (!SCM_NULLP (rest)) 
+           if (!scm_is_null (rest)) 
              {
                SCM next = handling_weaks
                  ? SCM_CDAR (l)
                  : SCM_CAR (l);
 
-               if (SCM_EQ_P (next, observer))
+               if (scm_is_eq (next, observer))
                  {
                    SCM_SETCDR (l, SCM_CDR (rest));
                    return;
@@ -721,7 +693,7 @@ core_environments_unobserve (SCM env, SCM observer)
              }
 
            l = rest;
-         } while (!SCM_NULLP (l));
+         } while (!scm_is_null (l));
        }
     }
 
@@ -729,20 +701,6 @@ core_environments_unobserve (SCM env, SCM observer)
 }
 
 
-static SCM
-core_environments_mark (SCM env)
-{
-  scm_gc_mark (CORE_ENVIRONMENT_OBSERVERS (env));
-  return CORE_ENVIRONMENT_WEAK_OBSERVER_VECTOR (env);
-}
-
-
-static void
-core_environments_finalize (SCM env)
-{
-}
-
-
 static void
 core_environments_preinit (struct core_environments_base *body)
 {
@@ -758,7 +716,7 @@ core_environments_init (struct core_environments_base *body,
 {
   body->funcs = funcs;
   body->observers = SCM_EOL;
-  body->weak_observers = scm_make_weak_value_hash_table (SCM_MAKINUM (1));
+  body->weak_observers = scm_make_weak_value_alist_vector (scm_from_int (1));
 }
 
 
@@ -800,9 +758,10 @@ update_catch_handler (void *ptr, SCM tag, SCM args)
 {
   struct update_data *data = (struct update_data *) ptr;
   SCM observer = data->observer;
-  SCM message = scm_makfrom0str ("Observer `~A' signals `~A' error: ~S");
+  SCM message =
+    scm_from_locale_string ("Observer `~A' signals `~A' error: ~S");
 
-  return scm_cons (message, scm_listify (observer, tag, args, SCM_UNDEFINED));
+  return scm_cons (message, scm_list_3 (observer, tag, args));
 }
 
 
@@ -819,7 +778,7 @@ core_environments_broadcast (SCM env)
        ? CORE_ENVIRONMENT_WEAK_OBSERVERS (env)
        : CORE_ENVIRONMENT_OBSERVERS (env);
 
-      for (; !SCM_NULLP (observers); observers = SCM_CDR (observers))
+      for (; !scm_is_null (observers); observers = SCM_CDR (observers))
        {
           struct update_data data;
          SCM observer = handling_weaks
@@ -839,7 +798,7 @@ core_environments_broadcast (SCM env)
        }
     }
 
-  if (!SCM_NULLP (errors))
+  if (!scm_is_null (errors))
     {
       /* Dirk:FIXME:: As soon as scm_misc_error is fixed to handle the name
        * parameter correctly it should not be necessary any more to also pass
@@ -892,14 +851,16 @@ leaf_environment_ref (SCM env, SCM sym)
 static SCM
 leaf_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM init)
 {
-  scm_sizet i;
+  size_t i;
   SCM result = init;
   SCM obarray = LEAF_ENVIRONMENT (env)->obarray;
 
-  for (i = 0; i < SCM_VECTOR_LENGTH (obarray); i++)
+  for (i = 0; i < SCM_HASHTABLE_N_BUCKETS (obarray); i++)
     {
       SCM l;
-      for (l = SCM_VELTS (obarray)[i]; !SCM_NULLP (l); l = SCM_CDR (l))
+      for (l = SCM_HASHTABLE_BUCKET (obarray, i);
+          !scm_is_null (l);
+          l = SCM_CDR (l))
        {
          SCM binding = SCM_CAR (l);
          SCM symbol = SCM_CAR (binding);
@@ -932,7 +893,7 @@ leaf_environment_undefine (SCM env, SCM sym)
   SCM obarray = LEAF_ENVIRONMENT (env)->obarray;
   SCM removed = obarray_remove (obarray, sym);
   
-  if (!SCM_FALSEP (removed))
+  if (scm_is_true (removed))
     core_environments_broadcast (env);
 
   return SCM_ENVIRONMENT_SUCCESS;
@@ -961,7 +922,7 @@ leaf_environment_set_x (SCM env, SCM sym, SCM val)
 
 
 static SCM
-leaf_environment_cell(SCM env, SCM sym, int for_write)
+leaf_environment_cell (SCM env, SCM sym, int for_write SCM_UNUSED)
 {
   SCM obarray = LEAF_ENVIRONMENT (env)->obarray;
   SCM binding = obarray_retrieve (obarray, sym);
@@ -969,32 +930,15 @@ leaf_environment_cell(SCM env, SCM sym, int for_write)
 }
 
 
-static SCM
-mark_leaf_environment (SCM env)
-{
-  scm_gc_mark (LEAF_ENVIRONMENT (env)->obarray);
-  return core_environments_mark (env);
-}
-
-
-static scm_sizet
-free_leaf_environment (SCM env)
-{
-  core_environments_finalize (env);
-
-  free (LEAF_ENVIRONMENT (env));
-  return sizeof (struct leaf_environment);
-}
-
 
 static int
-print_leaf_environment (SCM type, SCM port, scm_print_state *pstate)
+leaf_environment_print (SCM type, SCM port, scm_print_state *pstate SCM_UNUSED)
 {
-  SCM address = scm_ulong2num (SCM_UNPACK (type));
-  SCM base16 = scm_number_to_string (address, SCM_MAKINUM (16));
+  SCM address = scm_from_size_t (SCM_UNPACK (type));
+  SCM base16 = scm_number_to_string (address, scm_from_int (16));
 
   scm_puts ("#<leaf environment ", port);
-  scm_puts (SCM_STRING_CHARS (base16), port);
+  scm_display (base16, port);
   scm_puts (">", port);
 
   return 1;
@@ -1010,9 +954,7 @@ static struct scm_environment_funcs leaf_environment_funcs = {
   leaf_environment_cell,
   core_environments_observe,
   core_environments_unobserve,
-  mark_leaf_environment,
-  free_leaf_environment,
-  print_leaf_environment
+  leaf_environment_print
 };
 
 
@@ -1026,8 +968,8 @@ SCM_DEFINE (scm_make_leaf_environment, "make-leaf-environment", 0, 0, 0,
            "will be mutable.")
 #define FUNC_NAME s_scm_make_leaf_environment
 {
-  scm_sizet size = sizeof (struct leaf_environment);
-  struct leaf_environment *body = scm_must_malloc (size, FUNC_NAME);
+  size_t size = sizeof (struct leaf_environment);
+  struct leaf_environment *body = scm_gc_malloc (size, "leaf environment");
   SCM env;
 
   core_environments_preinit (&body->base);
@@ -1036,7 +978,7 @@ SCM_DEFINE (scm_make_leaf_environment, "make-leaf-environment", 0, 0, 0,
   env = scm_make_environment (body);
 
   core_environments_init (&body->base, &leaf_environment_funcs);
-  body->obarray = scm_make_vector (SCM_MAKINUM (scm_symhash_dim), SCM_EOL);  
+  body->obarray = scm_c_make_hash_table (DEFAULT_OBARRAY_SIZE);  
 
   return env;
 }
@@ -1045,10 +987,11 @@ SCM_DEFINE (scm_make_leaf_environment, "make-leaf-environment", 0, 0, 0,
 
 SCM_DEFINE (scm_leaf_environment_p, "leaf-environment?", 1, 0, 0, 
            (SCM object),
-           "Return #t if object is a leaf environment, or #f otherwise.")
+           "Return @code{#t} if object is a leaf environment, or @code{#f}\n"
+           "otherwise.")
 #define FUNC_NAME s_scm_leaf_environment_p
 {
-  return SCM_BOOL (SCM_LEAF_ENVIRONMENT_P (object));
+  return scm_from_bool (SCM_LEAF_ENVIRONMENT_P (object));
 }
 #undef FUNC_NAME
 
@@ -1095,9 +1038,9 @@ struct eval_environment {
 #define EVAL_ENVIRONMENT(env) \
   ((struct eval_environment *) SCM_CELL_WORD_1 (env))
 
-#define IMMUTABLE SCM_MAKINUM (0)
-#define MUTABLE   SCM_MAKINUM (1)
-#define UNKNOWN   SCM_MAKINUM (2)
+#define IMMUTABLE SCM_I_MAKINUM (0)
+#define MUTABLE   SCM_I_MAKINUM (1)
+#define UNKNOWN   SCM_I_MAKINUM (2)
 
 #define CACHED_LOCATION(x) SCM_CAR (x)
 #define CACHED_MUTABILITY(x) SCM_CADR (x)
@@ -1124,7 +1067,7 @@ eval_environment_lookup (SCM env, SCM sym, int for_write)
 
       SCM entry = SCM_CDR (binding);
 
-      if (SCM_CONSP (entry))
+      if (scm_is_pair (entry))
        {
          /* The entry in the obarray is a cached location. */
 
@@ -1135,15 +1078,15 @@ eval_environment_lookup (SCM env, SCM sym, int for_write)
            return location;
 
          mutability = CACHED_MUTABILITY (entry);
-         if (SCM_EQ_P (mutability, MUTABLE))
+         if (scm_is_eq (mutability, MUTABLE))
            return location;
 
-         if (SCM_EQ_P (mutability, UNKNOWN))
+         if (scm_is_eq (mutability, UNKNOWN))
            {
              SCM source_env = CACHED_SOURCE_ENVIRONMENT (entry);
              SCM location = SCM_ENVIRONMENT_CELL (source_env, sym, 1);
 
-             if (SCM_CONSP (location))
+             if (scm_is_pair (location))
                {
                  SET_CACHED_MUTABILITY (entry, MUTABLE);
                  return location;
@@ -1183,14 +1126,14 @@ eval_environment_lookup (SCM env, SCM sym, int for_write)
 
          if (!SCM_UNBNDP (location))
            {
-             if (SCM_CONSP (location))
+             if (scm_is_pair (location))
                {
                  SCM mutability = for_write ? MUTABLE : UNKNOWN;
                  SCM entry = scm_cons2 (location, mutability, source_env);
                  obarray_enter (obarray, sym, entry);
                  return location;
                }
-             else if (SCM_EQ_P (location, SCM_ENVIRONMENT_LOCATION_NO_CELL))
+             else if (scm_is_eq (location, SCM_ENVIRONMENT_LOCATION_NO_CELL))
                {
                  obarray_enter (obarray, sym, source_env);
                  return source_env;
@@ -1213,7 +1156,7 @@ eval_environment_ref (SCM env, SCM sym)
 {
   SCM location = eval_environment_lookup (env, sym, 0);
 
-  if (SCM_CONSP (location))
+  if (scm_is_pair (location))
     return SCM_CDR (location);
   else if (!SCM_UNBNDP (location))
     return SCM_ENVIRONMENT_REF (location, sym);
@@ -1231,7 +1174,7 @@ eval_environment_folder (SCM extended_data, SCM symbol, SCM value, SCM tail)
   if (!SCM_ENVIRONMENT_BOUND_P (local, symbol))
     {
       SCM proc_as_nr = SCM_CADR (extended_data);
-      unsigned long int proc_as_ul = scm_num2ulong (proc_as_nr, NULL, NULL);
+      unsigned long int proc_as_ul = scm_to_ulong (proc_as_nr);
       scm_environment_folder proc = (scm_environment_folder) proc_as_ul;
       SCM data = SCM_CDDR (extended_data);
 
@@ -1249,7 +1192,7 @@ eval_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM init)
 {
   SCM local = EVAL_ENVIRONMENT (env)->local;
   SCM imported = EVAL_ENVIRONMENT (env)->imported;
-  SCM proc_as_nr = scm_ulong2num ((unsigned long int) proc);
+  SCM proc_as_nr = scm_from_ulong ((unsigned long) proc);
   SCM extended_data = scm_cons2 (local, proc_as_nr, data);
   SCM tmp_result = scm_c_environment_fold (imported, eval_environment_folder, extended_data, init);
 
@@ -1283,7 +1226,7 @@ eval_environment_set_x (SCM env, SCM sym, SCM val)
 {
   SCM location = eval_environment_lookup (env, sym, 1);
 
-  if (SCM_CONSP (location))
+  if (scm_is_pair (location))
     {
       SCM_SETCDR (location, val);
       return SCM_ENVIRONMENT_SUCCESS;
@@ -1292,7 +1235,7 @@ eval_environment_set_x (SCM env, SCM sym, SCM val)
     {
       return SCM_ENVIRONMENT_SET (location, sym, val);
     }
-  else if (SCM_EQ_P (location, IMMUTABLE))
+  else if (scm_is_eq (location, IMMUTABLE))
     {
       return SCM_ENVIRONMENT_LOCATION_IMMUTABLE;
     }
@@ -1310,11 +1253,11 @@ eval_environment_cell (SCM env, SCM sym, int for_write)
 {
   SCM location = eval_environment_lookup (env, sym, for_write);
 
-  if (SCM_CONSP (location))
+  if (scm_is_pair (location))
     return location;
   else if (SCM_ENVIRONMENT_P (location))
     return SCM_ENVIRONMENT_LOCATION_NO_CELL;
-  else if (SCM_EQ_P (location, IMMUTABLE))
+  else if (scm_is_eq (location, IMMUTABLE))
     return SCM_ENVIRONMENT_LOCATION_IMMUTABLE;
   else
     return SCM_UNDEFINED;
@@ -1322,39 +1265,15 @@ eval_environment_cell (SCM env, SCM sym, int for_write)
 #undef FUNC_NAME
 
 
-static SCM
-mark_eval_environment (SCM env)
-{
-  struct eval_environment *body = EVAL_ENVIRONMENT (env);
-
-  scm_gc_mark (body->obarray);
-  scm_gc_mark (body->imported);
-  scm_gc_mark (body->imported_observer);
-  scm_gc_mark (body->local);
-  scm_gc_mark (body->local_observer);
-
-  return core_environments_mark (env);
-}
-
-
-static scm_sizet
-free_eval_environment (SCM env)
-{
-  core_environments_finalize (env);
-
-  free (EVAL_ENVIRONMENT (env));
-  return sizeof (struct eval_environment);
-}
-
 
 static int
-print_eval_environment (SCM type, SCM port, scm_print_state *pstate)
+eval_environment_print (SCM type, SCM port, scm_print_state *pstate SCM_UNUSED)
 {
-  SCM address = scm_ulong2num (SCM_UNPACK (type));
-  SCM base16 = scm_number_to_string (address, SCM_MAKINUM (16));
+  SCM address = scm_from_size_t (SCM_UNPACK (type));
+  SCM base16 = scm_number_to_string (address, scm_from_int (16));
 
   scm_puts ("#<eval environment ", port);
-  scm_puts (SCM_STRING_CHARS (base16), port);
+  scm_display (base16, port);
   scm_puts (">", port);
 
   return 1;
@@ -1370,9 +1289,7 @@ static struct scm_environment_funcs eval_environment_funcs = {
     eval_environment_cell,
     core_environments_observe,
     core_environments_unobserve,
-    mark_eval_environment,
-    free_eval_environment,
-    print_eval_environment
+    eval_environment_print
 };
 
 
@@ -1380,7 +1297,7 @@ void *scm_type_eval_environment = &eval_environment_funcs;
 
 
 static void
-eval_environment_observer (SCM caller, SCM eval_env)
+eval_environment_observer (SCM caller SCM_UNUSED, SCM eval_env)
 {
   SCM obarray = EVAL_ENVIRONMENT (eval_env)->obarray;
 
@@ -1392,22 +1309,26 @@ eval_environment_observer (SCM caller, SCM eval_env)
 SCM_DEFINE (scm_make_eval_environment, "make-eval-environment", 2, 0, 0, 
            (SCM local, SCM imported),
            "Return a new environment object eval whose bindings are the\n"
-           "union of the bindings in the environments local and imported,\n"
-           "with bindings from local taking precedence. Definitions made\n"
-           "in eval are placed in local.\n"
-           "Applying environment-define or environment-undefine to eval\n"
-           "has the same effect as applying the procedure to local.\n"
-           "Note that eval incorporates local and imported by reference:\n"
+           "union of the bindings in the environments @var{local} and\n"
+           "@var{imported}, with bindings from @var{local} taking\n"
+           "precedence. Definitions made in eval are placed in @var{local}.\n"
+           "Applying @code{environment-define} or\n"
+           "@code{environment-undefine} to eval has the same effect as\n"
+           "applying the procedure to @var{local}.\n"
+           "Note that eval incorporates @var{local} and @var{imported} by\n"
+           "reference:\n"
            "If, after creating eval, the program changes the bindings of\n"
-           "local or imported, those changes will be visible in eval.\n"
+           "@var{local} or @var{imported}, those changes will be visible\n"
+           "in eval.\n"
            "Since most Scheme evaluation takes place in eval environments,\n"
-           "they transparenty cache the bindings received from local and\n"
-           "imported. Thus, the first time the program looks up a symbol\n"
-           "in eval, eval may make calls to local or imported to find\n"
-           "their bindings, but subsequent references to that symbol will\n"
-           "be as fast as references to bindings in finite environments.\n"
-           "In typical use, local will be a finite environment, and\n"
-           "imported will be an import environment")
+           "they transparently cache the bindings received from @var{local}\n"
+           "and @var{imported}. Thus, the first time the program looks up\n"
+           "a symbol in eval, eval may make calls to @var{local} or\n"
+           "@var{imported} to find their bindings, but subsequent\n"
+           "references to that symbol will be as fast as references to\n"
+           "bindings in finite environments.\n"
+           "In typical use, @var{local} will be a finite environment, and\n"
+           "@var{imported} will be an import environment")
 #define FUNC_NAME s_scm_make_eval_environment
 {
   SCM env;
@@ -1416,7 +1337,7 @@ SCM_DEFINE (scm_make_eval_environment, "make-eval-environment", 2, 0, 0,
   SCM_ASSERT (SCM_ENVIRONMENT_P (local), local, SCM_ARG1, FUNC_NAME);
   SCM_ASSERT (SCM_ENVIRONMENT_P (imported), imported, SCM_ARG2, FUNC_NAME);
 
-  body = scm_must_malloc (sizeof (struct eval_environment), FUNC_NAME);
+  body = scm_gc_malloc (sizeof (struct eval_environment), "eval environment");
 
   core_environments_preinit (&body->base);
   body->obarray = SCM_BOOL_F;
@@ -1428,7 +1349,7 @@ SCM_DEFINE (scm_make_eval_environment, "make-eval-environment", 2, 0, 0,
   env = scm_make_environment (body);
 
   core_environments_init (&body->base, &eval_environment_funcs);
-  body->obarray = scm_make_vector (SCM_MAKINUM (scm_symhash_dim), SCM_EOL);  
+  body->obarray = scm_c_make_hash_table (DEFAULT_OBARRAY_SIZE);  
   body->imported = imported;
   body->imported_observer
     = SCM_ENVIRONMENT_OBSERVE (imported, eval_environment_observer, env, 1);
@@ -1443,17 +1364,18 @@ SCM_DEFINE (scm_make_eval_environment, "make-eval-environment", 2, 0, 0,
 
 SCM_DEFINE (scm_eval_environment_p, "eval-environment?", 1, 0, 0,
            (SCM object),
-           "Return #t if object is an eval environment, or #f otherwise.")
+           "Return @code{#t} if object is an eval environment, or @code{#f}\n"
+           "otherwise.")
 #define FUNC_NAME s_scm_eval_environment_p
 {
-  return SCM_BOOL (SCM_EVAL_ENVIRONMENT_P (object));
+  return scm_from_bool (SCM_EVAL_ENVIRONMENT_P (object));
 }
 #undef FUNC_NAME
 
 
 SCM_DEFINE (scm_eval_environment_local, "eval-environment-local", 1, 0, 0, 
            (SCM env),
-           "Return the local environment of eval environment env.")
+           "Return the local environment of eval environment @var{env}.")
 #define FUNC_NAME s_scm_eval_environment_local
 {
   SCM_ASSERT (SCM_EVAL_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -1465,7 +1387,7 @@ SCM_DEFINE (scm_eval_environment_local, "eval-environment-local", 1, 0, 0,
 
 SCM_DEFINE (scm_eval_environment_set_local_x, "eval-environment-set-local!", 2, 0, 0, 
            (SCM env, SCM local),
-           "Change env's local environment to LOCAL.")
+           "Change @var{env}'s local environment to @var{local}.")
 #define FUNC_NAME s_scm_eval_environment_set_local_x
 {
   struct eval_environment *body;
@@ -1491,7 +1413,7 @@ SCM_DEFINE (scm_eval_environment_set_local_x, "eval-environment-set-local!", 2,
 
 SCM_DEFINE (scm_eval_environment_imported, "eval-environment-imported", 1, 0, 0,
            (SCM env),
-           "Return the imported environment of eval environment env.")
+           "Return the imported environment of eval environment @var{env}.")
 #define FUNC_NAME s_scm_eval_environment_imported
 {
   SCM_ASSERT (SCM_EVAL_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -1503,7 +1425,7 @@ SCM_DEFINE (scm_eval_environment_imported, "eval-environment-imported", 1, 0, 0,
 
 SCM_DEFINE (scm_eval_environment_set_imported_x, "eval-environment-set-imported!", 2, 0, 0, 
            (SCM env, SCM imported),
-           "Change env's imported environment to IMPORTED.")
+           "Change @var{env}'s imported environment to @var{imported}.")
 #define FUNC_NAME s_scm_eval_environment_set_imported_x
 {
   struct eval_environment *body;
@@ -1565,7 +1487,7 @@ import_environment_lookup (SCM env, SCM sym)
   SCM result = SCM_UNDEFINED;
   SCM l;
 
-  for (l = imports; !SCM_NULLP (l); l = SCM_CDR (l))
+  for (l = imports; !scm_is_null (l); l = SCM_CDR (l))
     {
       SCM imported = SCM_CAR (l);
 
@@ -1573,14 +1495,14 @@ import_environment_lookup (SCM env, SCM sym)
        {
          if (SCM_UNBNDP (result))
            result = imported;
-         else if (SCM_CONSP (result))
+         else if (scm_is_pair (result))
            result = scm_cons (imported, result);
          else
            result = scm_cons2 (imported, result, SCM_EOL);
        }
     }
 
-  if (SCM_CONSP (result))
+  if (scm_is_pair (result))
     return scm_reverse (result);
   else
     return result;
@@ -1593,7 +1515,7 @@ import_environment_conflict (SCM env, SCM sym, SCM imports)
   SCM conflict_proc = IMPORT_ENVIRONMENT (env)->conflict_proc;
   SCM args = scm_cons2 (env, sym, scm_cons (imports, SCM_EOL));
 
-  return scm_apply (conflict_proc, args, SCM_EOL);
+  return scm_apply_0 (conflict_proc, args);
 }
 
 
@@ -1607,7 +1529,7 @@ import_environment_ref (SCM env, SCM sym)
     {
       return SCM_UNDEFINED;
     }
-  else if (SCM_CONSP (owner))
+  else if (scm_is_pair (owner))
     {
       SCM resolve = import_environment_conflict (env, sym, owner);
 
@@ -1632,11 +1554,11 @@ import_environment_folder (SCM extended_data, SCM symbol, SCM value, SCM tail)
   SCM imported_env = SCM_CADR (extended_data);
   SCM owner = import_environment_lookup (import_env, symbol);
   SCM proc_as_nr = SCM_CADDR (extended_data);
-  unsigned long int proc_as_ul = scm_num2ulong (proc_as_nr, NULL, NULL);
+  unsigned long int proc_as_ul = scm_to_ulong (proc_as_nr);
   scm_environment_folder proc = (scm_environment_folder) proc_as_ul;
   SCM data = SCM_CDDDR (extended_data);
 
-  if (SCM_CONSP (owner) && SCM_EQ_P (SCM_CAR (owner), imported_env))
+  if (scm_is_pair (owner) && scm_is_eq (SCM_CAR (owner), imported_env))
     owner = import_environment_conflict (import_env, symbol, owner);
 
   if (SCM_ENVIRONMENT_P (owner))
@@ -1650,11 +1572,11 @@ import_environment_folder (SCM extended_data, SCM symbol, SCM value, SCM tail)
 static SCM
 import_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM init)
 {
-  SCM proc_as_nr = scm_ulong2num ((unsigned long int) proc);
+  SCM proc_as_nr = scm_from_ulong ((unsigned long) proc);
   SCM result = init;
   SCM l;
 
-  for (l = IMPORT_ENVIRONMENT (env)->imports; !SCM_NULLP (l); l = SCM_CDR (l))
+  for (l = IMPORT_ENVIRONMENT (env)->imports; !scm_is_null (l); l = SCM_CDR (l))
     {
       SCM imported_env = SCM_CAR (l);
       SCM extended_data = scm_cons (env, scm_cons2 (imported_env, proc_as_nr, data));
@@ -1667,7 +1589,9 @@ import_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM ini
 
 
 static SCM
-import_environment_define (SCM env, SCM sym, SCM val)
+import_environment_define (SCM env SCM_UNUSED,
+                          SCM sym SCM_UNUSED,
+                          SCM val SCM_UNUSED)
 #define FUNC_NAME "import_environment_define"
 {
   return SCM_ENVIRONMENT_BINDING_IMMUTABLE;
@@ -1676,7 +1600,8 @@ import_environment_define (SCM env, SCM sym, SCM val)
 
 
 static SCM
-import_environment_undefine (SCM env, SCM sym)
+import_environment_undefine (SCM env SCM_UNUSED,
+                            SCM sym SCM_UNUSED)
 #define FUNC_NAME "import_environment_undefine"
 {
   return SCM_ENVIRONMENT_BINDING_IMMUTABLE;
@@ -1694,7 +1619,7 @@ import_environment_set_x (SCM env, SCM sym, SCM val)
     {
       return SCM_UNDEFINED;
     }
-  else if (SCM_CONSP (owner))
+  else if (scm_is_pair (owner))
     {
       SCM resolve = import_environment_conflict (env, sym, owner);
 
@@ -1721,7 +1646,7 @@ import_environment_cell (SCM env, SCM sym, int for_write)
     {
       return SCM_UNDEFINED;
     }
-  else if (SCM_CONSP (owner))
+  else if (scm_is_pair (owner))
     {
       SCM resolve = import_environment_conflict (env, sym, owner);
 
@@ -1738,34 +1663,16 @@ import_environment_cell (SCM env, SCM sym, int for_write)
 #undef FUNC_NAME
 
 
-static SCM
-mark_import_environment (SCM env)
-{
-  scm_gc_mark (IMPORT_ENVIRONMENT (env)->imports);
-  scm_gc_mark (IMPORT_ENVIRONMENT (env)->import_observers);
-  scm_gc_mark (IMPORT_ENVIRONMENT (env)->conflict_proc);
-  return core_environments_mark (env);
-}
-
-
-static scm_sizet
-free_import_environment (SCM env)
-{
-  core_environments_finalize (env);
-
-  free (IMPORT_ENVIRONMENT (env));
-  return sizeof (struct import_environment);
-}
-
 
 static int
-print_import_environment (SCM type, SCM port, scm_print_state *pstate)
+import_environment_print (SCM type, SCM port, 
+                         scm_print_state *pstate SCM_UNUSED)
 {
-  SCM address = scm_ulong2num (SCM_UNPACK (type));
-  SCM base16 = scm_number_to_string (address, SCM_MAKINUM (16));
+  SCM address = scm_from_size_t (SCM_UNPACK (type));
+  SCM base16 = scm_number_to_string (address, scm_from_int (16));
 
   scm_puts ("#<import environment ", port);
-  scm_puts (SCM_STRING_CHARS (base16), port);
+  scm_display (base16, port);
   scm_puts (">", port);
 
   return 1;
@@ -1781,9 +1688,7 @@ static struct scm_environment_funcs import_environment_funcs = {
   import_environment_cell,
   core_environments_observe,
   core_environments_unobserve,
-  mark_import_environment,
-  free_import_environment,
-  print_import_environment
+  import_environment_print
 };
 
 
@@ -1791,7 +1696,7 @@ void *scm_type_import_environment = &import_environment_funcs;
 
 
 static void
-import_environment_observer (SCM caller, SCM import_env)
+import_environment_observer (SCM caller SCM_UNUSED, SCM import_env)
 {
   core_environments_broadcast (import_env);
 }
@@ -1799,32 +1704,35 @@ import_environment_observer (SCM caller, SCM import_env)
 
 SCM_DEFINE (scm_make_import_environment, "make-import-environment", 2, 0, 0, 
            (SCM imports, SCM conflict_proc),
-           "Return a new environment imp whose bindings are the union of\n"
-           "the bindings from the environments in imports; imports must\n"
-           "be a list of environments. That is, imp binds symbol to\n"
-           "location when some element of imports does.\n"
-           "If two different elements of imports have a binding for the\n"
-           "same symbol, the conflict-proc is called with the following\n"
-           "parameters:  the import environment, the symbol and the list\n"
-           "of the imported environments that bind the symbol.  If the\n"
-           "conflict-proc returns an environment env, the conflict is\n"
-           "considered as resolved and the binding from env is used.  If\n"
-           "the conflict-proc returns some non-environment object, the\n"
-           "conflict is considered unresolved and the symbol is treated\n"
-           "as unspecified in the import environment.\n"
-           "The checking for conflicts may be performed lazily, i. e. at\m"
+           "Return a new environment @var{imp} whose bindings are the union\n"
+           "of the bindings from the environments in @var{imports};\n"
+           "@var{imports} must be a list of environments. That is,\n"
+           "@var{imp} binds a symbol to a location when some element of\n"
+           "@var{imports} does.\n"
+           "If two different elements of @var{imports} have a binding for\n"
+           "the same symbol, the @var{conflict-proc} is called with the\n"
+           "following parameters:  the import environment, the symbol and\n"
+           "the list of the imported environments that bind the symbol.\n"
+           "If the @var{conflict-proc} returns an environment @var{env},\n"
+           "the conflict is considered as resolved and the binding from\n"
+           "@var{env} is used.  If the @var{conflict-proc} returns some\n"
+           "non-environment object, the conflict is considered unresolved\n"
+           "and the symbol is treated as unspecified in the import\n"
+           "environment.\n"
+           "The checking for conflicts may be performed lazily, i. e. at\n"
            "the moment when a value or binding for a certain symbol is\n"
            "requested instead of the moment when the environment is\n"
            "created or the bindings of the imports change.\n"
-           "All bindings in imp are immutable. If you apply\n"
-           "environment-define or environment-undefine to imp, Guile\n"
-           "will signal an environment:immutable-binding error. However,\n"
-           "notice that the set of bindings in imp may still change, if\n"
-           "one of its imported environments changes.")
+           "All bindings in @var{imp} are immutable. If you apply\n"
+           "@code{environment-define} or @code{environment-undefine} to\n"
+           "@var{imp}, Guile will signal an\n"
+           " @code{environment:immutable-binding} error. However,\n"
+           "notice that the set of bindings in @var{imp} may still change,\n"
+           "if one of its imported environments changes.")
 #define FUNC_NAME s_scm_make_import_environment
 {
-  scm_sizet size = sizeof (struct import_environment);
-  struct import_environment *body = scm_must_malloc (size, FUNC_NAME);
+  size_t size = sizeof (struct import_environment);
+  struct import_environment *body = scm_gc_malloc (size, "import environment");
   SCM env;
 
   core_environments_preinit (&body->base);
@@ -1848,17 +1756,19 @@ SCM_DEFINE (scm_make_import_environment, "make-import-environment", 2, 0, 0,
 
 SCM_DEFINE (scm_import_environment_p, "import-environment?", 1, 0, 0, 
            (SCM object),
-           "Return #t if object is an import environment, or #f otherwise.")
+           "Return @code{#t} if object is an import environment, or\n"
+           "@code{#f} otherwise.")
 #define FUNC_NAME s_scm_import_environment_p
 {
-  return SCM_BOOL (SCM_IMPORT_ENVIRONMENT_P (object));
+  return scm_from_bool (SCM_IMPORT_ENVIRONMENT_P (object));
 }
 #undef FUNC_NAME
 
 
 SCM_DEFINE (scm_import_environment_imports, "import-environment-imports", 1, 0, 0, 
            (SCM env),
-           "Return the list of environments imported by the import environment env.")
+           "Return the list of environments imported by the import\n"
+           "environment @var{env}.")
 #define FUNC_NAME s_scm_import_environment_imports
 {
   SCM_ASSERT (SCM_IMPORT_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -1870,7 +1780,8 @@ SCM_DEFINE (scm_import_environment_imports, "import-environment-imports", 1, 0,
 
 SCM_DEFINE (scm_import_environment_set_imports_x, "import-environment-set-imports!", 2, 0, 0, 
            (SCM env, SCM imports),
-           "Change env's list of imported environments to imports, and check for conflicts.")
+           "Change @var{env}'s list of imported environments to\n"
+           "@var{imports}, and check for conflicts.")
 #define FUNC_NAME s_scm_import_environment_set_imports_x
 {
   struct import_environment *body = IMPORT_ENVIRONMENT (env);
@@ -1878,20 +1789,20 @@ SCM_DEFINE (scm_import_environment_set_imports_x, "import-environment-set-import
   SCM l;
 
   SCM_ASSERT (SCM_IMPORT_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
-  for (l = imports; SCM_CONSP (l); l = SCM_CDR (l))
+  for (l = imports; scm_is_pair (l); l = SCM_CDR (l))
     {
       SCM obj = SCM_CAR (l);
-      SCM_ASSERT (SCM_ENVIRONMENT_P (obj), imports, SCM_ARG1, FUNC_NAME);
+      SCM_ASSERT (SCM_ENVIRONMENT_P (obj), imports, SCM_ARG2, FUNC_NAME);
     }
-  SCM_ASSERT (SCM_NULLP (l), imports, SCM_ARG1, FUNC_NAME);
+  SCM_ASSERT (scm_is_null (l), imports, SCM_ARG2, FUNC_NAME);
 
-  for (l = body->import_observers; !SCM_NULLP (l); l = SCM_CDR (l))
+  for (l = body->import_observers; !scm_is_null (l); l = SCM_CDR (l))
     {
       SCM obs = SCM_CAR (l);
       SCM_ENVIRONMENT_UNOBSERVE (env, obs);
     }
 
-  for (l = imports; !SCM_NULLP (l); l = SCM_CDR (l))
+  for (l = imports; !scm_is_null (l); l = SCM_CDR (l))
     {
       SCM imp = SCM_CAR (l);
       SCM obs = SCM_ENVIRONMENT_OBSERVE (imp, import_environment_observer, env, 1);
@@ -1944,7 +1855,7 @@ export_environment_ref (SCM env, SCM sym)
   struct export_environment *body = EXPORT_ENVIRONMENT (env);
   SCM entry = scm_assq (sym, body->signature);
 
-  if (SCM_FALSEP (entry))
+  if (scm_is_false (entry))
     return SCM_UNDEFINED;
   else
     return SCM_ENVIRONMENT_REF (body->private, sym);
@@ -1959,7 +1870,7 @@ export_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM ini
   SCM result = init;
   SCM l;
 
-  for (l = body->signature; !SCM_NULLP (l); l = SCM_CDR (l))
+  for (l = body->signature; !scm_is_null (l); l = SCM_CDR (l))
     {
       SCM symbol = SCM_CAR (l);
       SCM value = SCM_ENVIRONMENT_REF (body->private, symbol);
@@ -1971,7 +1882,9 @@ export_environment_fold (SCM env, scm_environment_folder proc, SCM data, SCM ini
 
 
 static SCM
-export_environment_define (SCM env, SCM sym, SCM val)
+export_environment_define (SCM env SCM_UNUSED, 
+                          SCM sym SCM_UNUSED, 
+                          SCM val SCM_UNUSED)
 #define FUNC_NAME "export_environment_define"
 {
   return SCM_ENVIRONMENT_BINDING_IMMUTABLE;
@@ -1980,7 +1893,7 @@ export_environment_define (SCM env, SCM sym, SCM val)
 
 
 static SCM
-export_environment_undefine (SCM env, SCM sym)
+export_environment_undefine (SCM env SCM_UNUSED, SCM sym SCM_UNUSED)
 #define FUNC_NAME "export_environment_undefine"
 {
   return SCM_ENVIRONMENT_BINDING_IMMUTABLE;
@@ -1995,13 +1908,13 @@ export_environment_set_x (SCM env, SCM sym, SCM val)
   struct export_environment *body = EXPORT_ENVIRONMENT (env);
   SCM entry = scm_assq (sym, body->signature);
 
-  if (SCM_FALSEP (entry))
+  if (scm_is_false (entry))
     {
       return SCM_UNDEFINED;
     }
   else
     {
-      if (SCM_EQ_P (SCM_CADR (entry), symbol_mutable_location))
+      if (scm_is_eq (SCM_CADR (entry), symbol_mutable_location))
        return SCM_ENVIRONMENT_SET (body->private, sym, val);
       else
        return SCM_ENVIRONMENT_LOCATION_IMMUTABLE;
@@ -2017,13 +1930,13 @@ export_environment_cell (SCM env, SCM sym, int for_write)
   struct export_environment *body = EXPORT_ENVIRONMENT (env);
   SCM entry = scm_assq (sym, body->signature);
 
-  if (SCM_FALSEP (entry))
+  if (scm_is_false (entry))
     {
       return SCM_UNDEFINED;
     }
   else
     {
-      if (!for_write || SCM_EQ_P (SCM_CADR (entry), symbol_mutable_location))
+      if (!for_write || scm_is_eq (SCM_CADR (entry), symbol_mutable_location))
        return SCM_ENVIRONMENT_CELL (body->private, sym, for_write);
       else
        return SCM_ENVIRONMENT_LOCATION_IMMUTABLE;
@@ -2032,37 +1945,16 @@ export_environment_cell (SCM env, SCM sym, int for_write)
 #undef FUNC_NAME
 
 
-static SCM
-mark_export_environment (SCM env)
-{
-  struct export_environment *body = EXPORT_ENVIRONMENT (env);
-
-  scm_gc_mark (body->private);
-  scm_gc_mark (body->private_observer);
-  scm_gc_mark (body->signature);
-
-  return core_environments_mark (env);
-}
-
-
-static scm_sizet
-free_export_environment (SCM env)
-{
-  core_environments_finalize (env);
-
-  free (EXPORT_ENVIRONMENT (env));
-  return sizeof (struct export_environment);
-}
-
 
 static int
-print_export_environment (SCM type, SCM port, scm_print_state *pstate)
+export_environment_print (SCM type, SCM port,
+                         scm_print_state *pstate SCM_UNUSED)
 {
-  SCM address = scm_ulong2num (SCM_UNPACK (type));
-  SCM base16 = scm_number_to_string (address, SCM_MAKINUM (16));
+  SCM address = scm_from_size_t (SCM_UNPACK (type));
+  SCM base16 = scm_number_to_string (address, scm_from_int (16));
 
   scm_puts ("#<export environment ", port);
-  scm_puts (SCM_STRING_CHARS (base16), port);
+  scm_display (base16, port);
   scm_puts (">", port);
 
   return 1;
@@ -2078,9 +1970,7 @@ static struct scm_environment_funcs export_environment_funcs = {
   export_environment_cell,
   core_environments_observe,
   core_environments_unobserve,
-  mark_export_environment,
-  free_export_environment,
-  print_export_environment
+  export_environment_print
 };
 
 
@@ -2088,7 +1978,7 @@ void *scm_type_export_environment = &export_environment_funcs;
 
 
 static void
-export_environment_observer (SCM caller, SCM export_env)
+export_environment_observer (SCM caller SCM_UNUSED, SCM export_env)
 {
   core_environments_broadcast (export_env);
 }
@@ -2096,50 +1986,56 @@ export_environment_observer (SCM caller, SCM export_env)
 
 SCM_DEFINE (scm_make_export_environment, "make-export-environment", 2, 0, 0, 
            (SCM private, SCM signature),
-           "Return a new environment exp containing only those bindings\n"
-           "in private whose symbols are present in signature. The\n"
-           "private argument must be an environment.\n\n"
-           "The environment exp binds symbol to location when env does,\n"
-           "and symbol is exported by signature.\n\n"
-           "Signature is a list specifying which of the bindings in\n"
-           "private should be visible in exp. Each element of signature\n"
-           "should be a list of the form:\n"
+           "Return a new environment @var{exp} containing only those\n"
+           "bindings in private whose symbols are present in\n"
+           "@var{signature}. The @var{private} argument must be an\n"
+           "environment.\n\n"
+           "The environment @var{exp} binds symbol to location when\n"
+           "@var{env} does, and symbol is exported by @var{signature}.\n\n"
+           "@var{signature} is a list specifying which of the bindings in\n"
+           "@var{private} should be visible in @var{exp}. Each element of\n"
+           "@var{signature} should be a list of the form:\n"
            "  (symbol attribute ...)\n"
            "where each attribute is one of the following:\n"
-           "* the symbol mutable-location exp should treat the location\n"
-           "  bound to symbol as mutable. That is, exp will pass calls\n"
-           "  to env-set! or environment-cell directly through to\n"
-           "  private.\n"
-           "* the symbol immutable-location exp should treat the\n"
-           "  location bound to symbol as immutable. If the program\n"
-           "  applies environment-set! to exp and symbol, or calls\n"
-           "  environment-cell to obtain a writable value cell,\n"
-           "  environment-set! will signal an\n"
-           "  environment:immutable-location error. Note that, even if\n"
-           "  an export environment treats a location as immutable, the\n"
+           "@table @asis\n"
+           "@item the symbol @code{mutable-location}\n"
+           "  @var{exp} should treat the\n"
+           "  location bound to symbol as mutable. That is, @var{exp}\n"
+           "  will pass calls to @code{environment-set!} or\n"
+           "  @code{environment-cell} directly through to private.\n"
+           "@item the symbol @code{immutable-location}\n"
+           "  @var{exp} should treat\n"
+           "  the location bound to symbol as immutable. If the program\n"
+           "  applies @code{environment-set!} to @var{exp} and symbol, or\n"
+           "  calls @code{environment-cell} to obtain a writable value\n"
+           "  cell, @code{environment-set!} will signal an\n"
+           "  @code{environment:immutable-location} error. Note that, even\n"
+           "  if an export environment treats a location as immutable, the\n"
            "  underlying environment may treat it as mutable, so its\n"
            "  value may change.\n"
+           "@end table\n"
            "It is an error for an element of signature to specify both\n"
-           "mutable-location and immutable-location. If neither is\n"
-           "specified, immutable-location is assumed.\n\n"
+           "@code{mutable-location} and @code{immutable-location}. If\n"
+           "neither is specified, @code{immutable-location} is assumed.\n\n"
            "As a special case, if an element of signature is a lone\n"
-           "symbol sym, it is equivalent to an element of the form\n"
-           "(sym).\n\n"
-           "All bindings in exp are immutable. If you apply\n"
-           "environment-define or environment-undefine to exp, Guile\n"
-           "will signal an environment:immutable-binding error. However,\n"
-           "notice that the set of bindings in exp may still change, if\n"
-           "the bindings in private change.")
+           "symbol @var{sym}, it is equivalent to an element of the form\n"
+           "@code{(sym)}.\n\n"
+           "All bindings in @var{exp} are immutable. If you apply\n"
+           "@code{environment-define} or @code{environment-undefine} to\n"
+           "@var{exp}, Guile will signal an\n"
+           "@code{environment:immutable-binding} error. However,\n"
+           "notice that the set of bindings in @var{exp} may still change,\n"
+           "if the bindings in private change.")
 #define FUNC_NAME s_scm_make_export_environment
 {
-  scm_sizet size;
+  size_t size;
   struct export_environment *body;
   SCM env;
 
   SCM_ASSERT (SCM_ENVIRONMENT_P (private), private, SCM_ARG1, FUNC_NAME);
 
   size = sizeof (struct export_environment);
-  body = scm_must_malloc (size, FUNC_NAME);
+  body = scm_gc_malloc (size, "export environment");
 
   core_environments_preinit (&body->base);
   body->private = SCM_BOOL_F;
@@ -2163,17 +2059,18 @@ SCM_DEFINE (scm_make_export_environment, "make-export-environment", 2, 0, 0,
 
 SCM_DEFINE (scm_export_environment_p, "export-environment?", 1, 0, 0, 
            (SCM object),
-           "Return #t if object is an export environment, or #f otherwise.")
+           "Return @code{#t} if object is an export environment, or\n"
+           "@code{#f} otherwise.")
 #define FUNC_NAME s_scm_export_environment_p
 {
-  return SCM_BOOL (SCM_EXPORT_ENVIRONMENT_P (object));
+  return scm_from_bool (SCM_EXPORT_ENVIRONMENT_P (object));
 }
 #undef FUNC_NAME
 
 
 SCM_DEFINE (scm_export_environment_private, "export-environment-private", 1, 0, 0, 
            (SCM env),
-           "Return the private environment of export environment env.")
+           "Return the private environment of export environment @var{env}.")
 #define FUNC_NAME s_scm_export_environment_private
 {
   SCM_ASSERT (SCM_EXPORT_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -2185,7 +2082,7 @@ SCM_DEFINE (scm_export_environment_private, "export-environment-private", 1, 0,
 
 SCM_DEFINE (scm_export_environment_set_private_x, "export-environment-set-private!", 2, 0, 0, 
            (SCM env, SCM private),
-           "Change the private environment of export environment env.")
+           "Change the private environment of export environment @var{env}.")
 #define FUNC_NAME s_scm_export_environment_set_private_x
 {
   struct export_environment *body;
@@ -2207,7 +2104,7 @@ SCM_DEFINE (scm_export_environment_set_private_x, "export-environment-set-privat
 
 SCM_DEFINE (scm_export_environment_signature, "export-environment-signature", 1, 0, 0, 
            (SCM env),
-           "Return the signature of export environment env.")
+           "Return the signature of export environment @var{env}.")
 #define FUNC_NAME s_scm_export_environment_signature
 {
   SCM_ASSERT (SCM_EXPORT_ENVIRONMENT_P (env), env, SCM_ARG1, FUNC_NAME);
@@ -2223,11 +2120,11 @@ export_environment_parse_signature (SCM signature, const char* caller)
   SCM result = SCM_EOL;
   SCM l;
 
-  for (l = signature; SCM_CONSP (l); l = SCM_CDR (l))
+  for (l = signature; scm_is_pair (l); l = SCM_CDR (l))
     {
       SCM entry = SCM_CAR (l);
 
-      if (SCM_SYMBOLP (entry))
+      if (scm_is_symbol (entry))
        {
          SCM new_entry = scm_cons2 (entry, symbol_immutable_location, SCM_EOL);
          result = scm_cons (new_entry, result);
@@ -2241,22 +2138,22 @@ export_environment_parse_signature (SCM signature, const char* caller)
          SCM mutability;
          SCM l2;
 
-         SCM_ASSERT (SCM_CONSP (entry), entry, SCM_ARGn, caller);
-         SCM_ASSERT (SCM_SYMBOLP (SCM_CAR (entry)), entry, SCM_ARGn, caller);
+         SCM_ASSERT (scm_is_pair (entry), entry, SCM_ARGn, caller);
+         SCM_ASSERT (scm_is_symbol (SCM_CAR (entry)), entry, SCM_ARGn, caller);
 
          sym = SCM_CAR (entry);
 
-         for (l2 = SCM_CDR (entry); SCM_CONSP (l2); l2 = SCM_CDR (l2))
+         for (l2 = SCM_CDR (entry); scm_is_pair (l2); l2 = SCM_CDR (l2))
            {
              SCM attribute = SCM_CAR (l2);
-             if (SCM_EQ_P (attribute, symbol_immutable_location))
+             if (scm_is_eq (attribute, symbol_immutable_location))
                immutable = 1;
-             else if (SCM_EQ_P (attribute, symbol_mutable_location))
+             else if (scm_is_eq (attribute, symbol_mutable_location))
                mutable = 1;
              else
                SCM_ASSERT (0, entry, SCM_ARGn, caller);
            }
-         SCM_ASSERT (SCM_NULLP (l2), entry, SCM_ARGn, caller);
+         SCM_ASSERT (scm_is_null (l2), entry, SCM_ARGn, caller);
          SCM_ASSERT (!mutable || !immutable, entry, SCM_ARGn, caller);
 
          if (!mutable && !immutable)
@@ -2267,7 +2164,7 @@ export_environment_parse_signature (SCM signature, const char* caller)
          result = scm_cons (new_entry, result);
        }
     }
-  SCM_ASSERT (SCM_NULLP (l), signature, SCM_ARGn, caller);
+  SCM_ASSERT (scm_is_null (l), signature, SCM_ARGn, caller);
 
   /* Dirk:FIXME:: Now we know that signature is syntactically correct.  There
    * are, however, no checks for symbols entered twice with contradicting
@@ -2281,7 +2178,7 @@ export_environment_parse_signature (SCM signature, const char* caller)
 
 SCM_DEFINE (scm_export_environment_set_signature_x, "export-environment-set-signature!", 2, 0, 0, 
            (SCM env, SCM signature),
-           "Change the signature of export environment env.")
+           "Change the signature of export environment @var{env}.")
 #define FUNC_NAME s_scm_export_environment_set_signature_x
 {
   SCM parsed_sig;
@@ -2302,15 +2199,15 @@ scm_environments_prehistory ()
 {
   /* create environment smob */
   scm_tc16_environment = scm_make_smob_type ("environment", 0);
-  scm_set_smob_mark (scm_tc16_environment, mark_environment);
-  scm_set_smob_free (scm_tc16_environment, free_environment);
-  scm_set_smob_print (scm_tc16_environment, print_environment);
+  scm_set_smob_print (scm_tc16_environment, environment_print);
 
   /* create observer smob */
   scm_tc16_observer = scm_make_smob_type ("observer", 0);
-  scm_set_smob_mark (scm_tc16_observer, mark_observer);
-  scm_set_smob_free (scm_tc16_observer, free_observer);
-  scm_set_smob_print (scm_tc16_observer, print_observer);
+  scm_set_smob_print (scm_tc16_observer, observer_print);
+
+  /* create system environment */
+  scm_system_environment = scm_make_leaf_environment ();
+  scm_permanent_object (scm_system_environment);
 }