merge from 1.8 branch
[bpt/guile.git] / ice-9 / boot-9.scm
index 5fd5b63..0f56f98 100644 (file)
@@ -1,46 +1,23 @@
 ;;; installed-scm-file
 
-;;;; Copyright (C) 1995,1996,1997,1998,1999,2000,2001,2002 Free Software Foundation, Inc.
+;;;; Copyright (C) 1995,1996,1997,1998,1999,2000,2001,2002,2003,2004,2005,2006
+;;;; 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 program is distributed in the hope that it will be useful,
+;;;; 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 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.
-;;;;
-;;;; 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.
+;;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+;;;; Lesser General Public License for more details.
+;;;; 
+;;;; You should have received a copy of the GNU Lesser General Public
+;;;; License along with this library; if not, write to the Free Software
+;;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 ;;;;
+
 \f
 
 ;;; Commentary:
 ;;; Code:
 
 \f
-;;; {Deprecation}
-;;;
-
-;; We don't have macros here, but we do want to define
-;; `begin-deprecated' early.
-
-(define begin-deprecated
-  (procedure->memoizing-macro
-   (lambda (exp env)
-     (if (include-deprecated-features)
-        `(begin ,@(cdr exp))
-        `#f))))
 
-\f
 ;;; {Features}
-;;
+;;;
 
 (define (provide sym)
   (if (not (memq sym *features*))
       (set! *features* (cons sym *features*))))
 
-;;; Return #t iff FEATURE is available to this Guile interpreter.
-;;; In SLIB, provided? also checks to see if the module is available.
-;;; We should do that too, but don't.
+;; Return #t iff FEATURE is available to this Guile interpreter.  In SLIB,
+;; provided? also checks to see if the module is available.  We should do that
+;; too, but don't.
+
 (define (provided? feature)
   (and (memq feature *features*) #t))
 
-(begin-deprecated
- (define feature? provided?))
+;; let format alias simple-format until the more complete version is loaded
 
-;;; let format alias simple-format until the more complete version is loaded
 (define format simple-format)
 
+;; this is scheme wrapping the C code so the final pred call is a tail call,
+;; per SRFI-13 spec
+(define (string-any char_pred s . rest)
+  (let ((start (if (null? rest)
+                  0 (car rest)))
+       (end   (if (or (null? rest) (null? (cdr rest)))
+                  (string-length s) (cadr rest))))
+    (if (and (procedure? char_pred)
+            (> end start)
+            (<= end (string-length s))) ;; let c-code handle range error
+       (or (string-any-c-code char_pred s start (1- end))
+           (char_pred (string-ref s (1- end))))
+       (string-any-c-code char_pred s start end))))
+
+;; this is scheme wrapping the C code so the final pred call is a tail call,
+;; per SRFI-13 spec
+(define (string-every char_pred s . rest)
+  (let ((start (if (null? rest)
+                  0 (car rest)))
+       (end   (if (or (null? rest) (null? (cdr rest)))
+                  (string-length s) (cadr rest))))
+    (if (and (procedure? char_pred)
+            (> end start)
+            (<= end (string-length s))) ;; let c-code handle range error
+       (and (string-every-c-code char_pred s start (1- end))
+            (char_pred (string-ref s (1- end))))
+       (string-every-c-code char_pred s start end))))
+
+;; A variant of string-fill! that we keep for compatability
+;;
+(define (substring-fill! str start end fill)
+  (string-fill! str fill start end))
+
+\f
+
+;;; {EVAL-CASE}
+;;;
+
+;; (eval-case ((situation*) forms)* (else forms)?)
+;;
+;; Evaluate certain code based on the situation that eval-case is used
+;; in.  The only defined situation right now is `load-toplevel' which
+;; triggers for code evaluated at the top-level, for example from the
+;; REPL or when loading a file.
+
+(define eval-case
+  (procedure->memoizing-macro
+   (lambda (exp env)
+     (define (toplevel-env? env)
+       (or (not (pair? env)) (not (pair? (car env)))))
+     (define (syntax)
+       (error "syntax error in eval-case"))
+     (let loop ((clauses (cdr exp)))
+       (cond
+       ((null? clauses)
+        #f)
+       ((not (list? (car clauses)))
+        (syntax))
+       ((eq? 'else (caar clauses))
+        (or (null? (cdr clauses))
+            (syntax))
+        (cons 'begin (cdar clauses)))
+       ((not (list? (caar clauses)))
+        (syntax))
+       ((and (toplevel-env? env)
+             (memq 'load-toplevel (caar clauses)))
+        (cons 'begin (cdar clauses)))
+       (else
+        (loop (cdr clauses))))))))
+
+\f
+
+;;; {Defmacros}
+;;;
+;;; Depends on: features, eval-case
+;;;
+
+(define macro-table (make-weak-key-hash-table 61))
+(define xformer-table (make-weak-key-hash-table 61))
+
+(define (defmacro? m)  (hashq-ref macro-table m))
+(define (assert-defmacro?! m) (hashq-set! macro-table m #t))
+(define (defmacro-transformer m) (hashq-ref xformer-table m))
+(define (set-defmacro-transformer! m t) (hashq-set! xformer-table m t))
+
+(define defmacro:transformer
+  (lambda (f)
+    (let* ((xform (lambda (exp env)
+                   (copy-tree (apply f (cdr exp)))))
+          (a (procedure->memoizing-macro xform)))
+      (assert-defmacro?! a)
+      (set-defmacro-transformer! a f)
+      a)))
+
+
+(define defmacro
+  (let ((defmacro-transformer
+         (lambda (name parms . body)
+           (let ((transformer `(lambda ,parms ,@body)))
+             `(eval-case
+               ((load-toplevel)
+                (define ,name (defmacro:transformer ,transformer)))
+               (else
+                (error "defmacro can only be used at the top level")))))))
+    (defmacro:transformer defmacro-transformer)))
+
+(define defmacro:syntax-transformer
+  (lambda (f)
+    (procedure->syntax
+             (lambda (exp env)
+               (copy-tree (apply f (cdr exp)))))))
+
+
+;; XXX - should the definition of the car really be looked up in the
+;; current module?
+
+(define (macroexpand-1 e)
+  (cond
+   ((pair? e) (let* ((a (car e))
+                    (val (and (symbol? a) (local-ref (list a)))))
+               (if (defmacro? val)
+                   (apply (defmacro-transformer val) (cdr e))
+                   e)))
+   (#t e)))
+
+(define (macroexpand e)
+  (cond
+   ((pair? e) (let* ((a (car e))
+                    (val (and (symbol? a) (local-ref (list a)))))
+               (if (defmacro? val)
+                   (macroexpand (apply (defmacro-transformer val) (cdr e)))
+                   e)))
+   (#t e)))
+
+(provide 'defmacro)
+
+\f
+
+;;; {Deprecation}
+;;;
+;;; Depends on: defmacro
+;;;
+
+(defmacro begin-deprecated forms
+  (if (include-deprecated-features)
+      (cons begin forms)
+      #f))
+
 \f
+
 ;;; {R4RS compliance}
+;;;
 
 (primitive-load-path "ice-9/r4rs.scm")
 
 \f
-;;; {Simple Debugging Tools}
-;;
 
+;;; {Simple Debugging Tools}
+;;;
 
 ;; peek takes any number of arguments, writes them to the
 ;; current ouput port, and returns the last argument.
       (car (last-pair stuff)))))
 
 \f
+
 ;;; {Trivial Functions}
 ;;;
 
 (define (identity x) x)
-(define (1+ n) (+ n 1))
-(define (1- n) (+ n -1))
 (define (and=> value procedure) (and value (procedure value)))
-(define (make-hash-table k) (make-vector k '()))
+(define call/cc call-with-current-continuation)
 
 ;;; apply-to-args is functionally redundant with apply and, worse,
 ;;; is less general than apply since it only takes two arguments.
 
 (define (apply-to-args args fn) (apply fn args))
 
+(defmacro false-if-exception (expr)
+  `(catch #t (lambda () ,expr)
+         (lambda args #f)))
+
 \f
 
-;;; {Integer Math}
+;;; {General Properties}
 ;;;
 
-(define (ipow-by-squaring x k acc proc)
-  (cond ((zero? k) acc)
-       ((= 1 k) (proc acc x))
-       (else (ipow-by-squaring (proc x x)
-                               (quotient k 2)
-                               (if (even? k) acc (proc acc x))
-                               proc))))
+;; This is a more modern interface to properties.  It will replace all
+;; other property-like things eventually.
+
+(define (make-object-property)
+  (let ((prop (primitive-make-property #f)))
+    (make-procedure-with-setter
+     (lambda (obj) (primitive-property-ref prop obj))
+     (lambda (obj val) (primitive-property-set! prop obj val)))))
 
 \f
+
 ;;; {Symbol Properties}
 ;;;
 
     (if pair
        (symbol-pset! sym (delq! pair (symbol-pref sym))))))
 
-;;; {General Properties}
-;;;
-
-;; This is a more modern interface to properties.  It will replace all
-;; other property-like things eventually.
-
-(define (make-object-property)
-  (let ((prop (primitive-make-property #f)))
-    (make-procedure-with-setter
-     (lambda (obj) (primitive-property-ref prop obj))
-     (lambda (obj val) (primitive-property-set! prop obj val)))))
-
 \f
 
 ;;; {Arrays}
 ;;;
 
-(if (provided? 'array)
-    (primitive-load-path "ice-9/arrays.scm"))
+(define (array-shape a)
+  (map (lambda (ind) (if (number? ind) (list 0 (+ -1 ind)) ind))
+       (array-dimensions a)))
 
 \f
+
 ;;; {Keywords}
 ;;;
 
-(define (symbol->keyword symbol)
-  (make-keyword-from-dash-symbol (symbol-append '- symbol)))
-
-(define (keyword->symbol kw)
-  (let ((sym (symbol->string (keyword-dash-symbol kw))))
-    (string->symbol (substring sym 1 (string-length sym)))))
-
 (define (kw-arg-ref args kw)
   (let ((rem (member kw args)))
     (and rem (pair? (cdr rem)) (cadr rem))))
 \f
 
 ;;; {Structs}
+;;;
 
 (define (struct-layout s)
   (struct-ref (struct-vtable s) vtable-index-layout))
 
 \f
 
-;;; Environments
+;;; {Environments}
+;;;
 
 (define the-environment
   (procedure->syntax
 
 (define (environment-module env)
   (let ((closure (and (pair? env) (car (last-pair env)))))
-    (and closure (procedure-property closure 'module))))
+    (and closure (eval-closure-module closure))))
 
 \f
+
 ;;; {Records}
 ;;;
 
 (provide 'record)
 
 \f
+
 ;;; {Booleans}
 ;;;
 
 (define (->bool x) (not (not x)))
 
 \f
+
 ;;; {Symbols}
 ;;;
 
   (string->symbol (apply string args)))
 
 \f
+
 ;;; {Lists}
 ;;;
 
             n
             (loop (+ n 1) (cdr l))))))
 
-(define (make-list n . init)
-  (if (pair? init) (set! init (car init)))
-  (let loop ((answer '())
-            (n n))
-    (if (<= n 0)
-       answer
-       (loop (cons init answer) (- n 1)))))
-
 \f
+
 ;;; {and-map and or-map}
 ;;;
 ;;; (and-map fn lst) is like (and (fn (car lst)) (fn (cadr lst)) (fn...) ...)
 (if (provided? 'socket)
     (primitive-load-path "ice-9/networking.scm"))
 
+;; For reference, Emacs file-exists-p uses stat in this same way.
+;; ENHANCE-ME: Catching an exception from stat is a bit wasteful, do this in
+;; C where all that's needed is to inspect the return from stat().
 (define file-exists?
   (if (provided? 'posix)
       (lambda (str)
-       (access? str F_OK))
+       (->bool (false-if-exception (stat str))))
       (lambda (str)
        (let ((port (catch 'system-error (lambda () (open-file str OPEN_READ))
                           (lambda args #f))))
       #f))
 
 \f
+
 ;;; {Error Handling}
 ;;;
 
   (putenv name))
 
 \f
+
 ;;; {Load Paths}
 ;;;
 
                   file)))
 
 \f
+
 ;;; {Help for scm_shell}
+;;;
 ;;; The argument-processing code used by Guile-based shells generates
 ;;; Scheme code based on the argument list.  This page contains help
 ;;; functions for the code it generates.
+;;;
 
 (define (command-line) (program-arguments))
 
        (primitive-load init-file))))
 
 \f
+
 ;;; {Loading by paths}
+;;;
 
 ;;; Load a Scheme source file named NAME, searching for it in the
 ;;; directories listed in %load-path, and applying each of the file
 
 
 \f
+
 ;;; {Transcendental Functions}
 ;;;
 ;;; Derived from "Transcen.scm", Complex trancendental functions for SCM.
 (define (sqrt z)
   (if (real? z)
       (if (negative? z) (make-rectangular 0 ($sqrt (- z)))
-         ($sqrt z))
+          ($sqrt z))
       (make-polar ($sqrt (magnitude z)) (/ (angle z) 2))))
 
 (define expt
   (let ((integer-expt integer-expt))
     (lambda (z1 z2)
-      (cond ((integer? z2)
-            (if (>= z2 0)
-                (integer-expt z1 z2)
-                (/ 1 (integer-expt z1 (- z2)))))
+      (cond ((and (exact? z2) (integer? z2))
+            (integer-expt z1 z2))
            ((and (real? z2) (real? z1) (>= z1 0))
             ($expt z1 z2))
            (else
 
 ;;; {Reader Extensions}
 ;;;
-
 ;;; Reader code for various "#c" forms.
 ;;;
 
                          "#. read expansion found and read-eval? is #f."))))
 
 \f
+
 ;;; {Command Line Options}
 ;;;
 
                (left-part (string-append
                            (with-output-to-string
                              (lambda ()
-                               (map (lambda (x) (display (keyword-symbol x)) (display " "))
+                               (map (lambda (x) (display (keyword->symbol x)) (display " "))
                                     opts-proper)))
                            arg-name))
                (middle-part (if (and (< (string-length left-part) 30)
 ;;;
 ;;; These are the low level data structures for modules.
 ;;;
+;;; Every module object is of the type 'module-type', which is a record
+;;; consisting of the following members:
+;;;
+;;; - eval-closure: the function that defines for its module the strategy that
+;;;   shall be followed when looking up symbols in the module.
+;;;
+;;;   An eval-closure is a function taking two arguments: the symbol to be
+;;;   looked up and a boolean value telling whether a binding for the symbol
+;;;   should be created if it does not exist yet.  If the symbol lookup
+;;;   succeeded (either because an existing binding was found or because a new
+;;;   binding was created), a variable object representing the binding is
+;;;   returned.  Otherwise, the value #f is returned.  Note that the eval
+;;;   closure does not take the module to be searched as an argument: During
+;;;   construction of the eval-closure, the eval-closure has to store the
+;;;   module it belongs to in its environment.  This means, that any
+;;;   eval-closure can belong to only one module.
+;;;
+;;;   The eval-closure of a module can be defined arbitrarily.  However, three
+;;;   special cases of eval-closures are to be distinguished: During startup
+;;;   the module system is not yet activated.  In this phase, no modules are
+;;;   defined and all bindings are automatically stored by the system in the
+;;;   pre-modules-obarray.  Since no eval-closures exist at this time, the
+;;;   functions which require an eval-closure as their argument need to be
+;;;   passed the value #f.
+;;;
+;;;   The other two special cases of eval-closures are the
+;;;   standard-eval-closure and the standard-interface-eval-closure.  Both
+;;;   behave equally for the case that no new binding is to be created.  The
+;;;   difference between the two comes in, when the boolean argument to the
+;;;   eval-closure indicates that a new binding shall be created if it is not
+;;;   found.
+;;;
+;;;   Given that no new binding shall be created, both standard eval-closures
+;;;   define the following standard strategy of searching bindings in the
+;;;   module: First, the module's obarray is searched for the symbol.  Second,
+;;;   if no binding for the symbol was found in the module's obarray, the
+;;;   module's binder procedure is exececuted.  If this procedure did not
+;;;   return a binding for the symbol, the modules referenced in the module's
+;;;   uses list are recursively searched for a binding of the symbol.  If the
+;;;   binding can not be found in these modules also, the symbol lookup has
+;;;   failed.
+;;;
+;;;   If a new binding shall be created, the standard-interface-eval-closure
+;;;   immediately returns indicating failure.  That is, it does not even try
+;;;   to look up the symbol.  In contrast, the standard-eval-closure would
+;;;   first search the obarray, and if no binding was found there, would
+;;;   create a new binding in the obarray, therefore not calling the binder
+;;;   procedure or searching the modules in the uses list.
+;;;
+;;;   The explanation of the following members obarray, binder and uses
+;;;   assumes that the symbol lookup follows the strategy that is defined in
+;;;   the standard-eval-closure and the standard-interface-eval-closure.
+;;;
+;;; - obarray: a hash table that maps symbols to variable objects.  In this
+;;;   hash table, the definitions are found that are local to the module (that
+;;;   is, not imported from other modules).  When looking up bindings in the
+;;;   module, this hash table is searched first.
+;;;
+;;; - binder: either #f or a function taking a module and a symbol argument.
+;;;   If it is a function it is called after the obarray has been
+;;;   unsuccessfully searched for a binding.  It then can provide bindings
+;;;   that would otherwise not be found locally in the module.
+;;;
+;;; - uses: a list of modules from which non-local bindings can be inherited.
+;;;   These modules are the third place queried for bindings after the obarray
+;;;   has been unsuccessfully searched and the binder function did not deliver
+;;;   a result either.
+;;;
+;;; - transformer: either #f or a function taking a scheme expression as
+;;;   delivered by read.  If it is a function, it will be called to perform
+;;;   syntax transformations (e. g. makro expansion) on the given scheme
+;;;   expression. The output of the transformer function will then be passed
+;;;   to Guile's internal memoizer.  This means that the output must be valid
+;;;   scheme code.  The only exception is, that the output may make use of the
+;;;   syntax extensions provided to identify the modules that a binding
+;;;   belongs to.
+;;;
+;;; - name: the name of the module.  This is used for all kinds of printing
+;;;   outputs.  In certain places the module name also serves as a way of
+;;;   identification.  When adding a module to the uses list of another
+;;;   module, it is made sure that the new uses list will not contain two
+;;;   modules of the same name.
+;;;
+;;; - kind: classification of the kind of module.  The value is (currently?)
+;;;   only used for printing.  It has no influence on how a module is treated.
+;;;   Currently the following values are used when setting the module kind:
+;;;   'module, 'directory, 'interface, 'custom-interface.  If no explicit kind
+;;;   is set, it defaults to 'module.
+;;;
+;;; - duplicates-handlers
+;;;
+;;; - duplicates-interface
+;;;
+;;; - observers
+;;;
+;;; - weak-observers
+;;;
+;;; - observer-id
+;;;
+;;; In addition, the module may (must?) contain a binding for
+;;; %module-public-interface... More explanations here...
+;;;
 ;;; !!! warning: The interface to lazy binder procedures is going
 ;;; to be changed in an incompatible way to permit all the basic
 ;;; module ops to be virtualized.
 ;;;
 
 \f
+
 ;;; {Printing Modules}
+;;;
+
 ;; This is how modules are printed.  You can re-define it.
 ;; (Redefining is actually more complicated than simply redefining
 ;; %print-module because that would only change the binding and not
 ;; is a (CLOSURE module symbol) which, as a last resort, can provide
 ;; bindings that would otherwise not be found locally in the module.
 ;;
-;; NOTE: If you change here, you also need to change libguile/modules.h.
+;; NOTE: If you change anything here, you also need to change
+;; libguile/modules.h.
 ;;
 (define module-type
   (make-record-type 'module
                    '(obarray uses binder eval-closure transformer name kind
-                             observers weak-observers observer-id)
+                     duplicates-handlers duplicates-interface
+                     observers weak-observers observer-id)
                    %print-module))
 
 ;; make-module &opt size uses binder
       (if (> (length args) 3)
          (error "Too many args to make-module." args))
 
-      (let ((size (parse-arg 0 1021))
+      (let ((size (parse-arg 0 31))
            (uses (parse-arg 1 '()))
            (binder (parse-arg 2 #f)))
 
            (error
             "Lazy-binder expected to be a procedure or #f." binder))
 
-       (let ((module (module-constructor (make-vector size '())
-                                         uses binder #f #f #f #f
+       (let ((module (module-constructor (make-hash-table size)
+                                         uses binder #f #f #f #f #f #f
                                          '()
                                          (make-weak-value-hash-table 31)
                                          0)))
 (define set-module-name! (record-modifier module-type 'name))
 (define module-kind (record-accessor module-type 'kind))
 (define set-module-kind! (record-modifier module-type 'kind))
+(define module-duplicates-handlers
+  (record-accessor module-type 'duplicates-handlers))
+(define set-module-duplicates-handlers!
+  (record-modifier module-type 'duplicates-handlers))
+(define module-duplicates-interface
+  (record-accessor module-type 'duplicates-interface))
+(define set-module-duplicates-interface!
+  (record-modifier module-type 'duplicates-interface))
 (define module-observers (record-accessor module-type 'observers))
 (define set-module-observers! (record-modifier module-type 'observers))
 (define module-weak-observers (record-accessor module-type 'weak-observers))
       (set-procedure-property! closure 'module module))))
 
 \f
+
 ;;; {Observer protocol}
 ;;;
 
        (set-module-observers! module (delq1! id (module-observers module)))))
   *unspecified*)
 
+(define module-defer-observers #f)
+(define module-defer-observers-mutex (make-mutex))
+(define module-defer-observers-table (make-hash-table))
+
 (define (module-modified m)
+  (if module-defer-observers
+      (hash-set! module-defer-observers-table m #t)
+      (module-call-observers m)))
+
+;;; This function can be used to delay calls to observers so that they
+;;; can be called once only in the face of massive updating of modules.
+;;;
+(define (call-with-deferred-observers thunk)
+  (dynamic-wind
+      (lambda ()
+       (lock-mutex module-defer-observers-mutex)
+       (set! module-defer-observers #t))
+      thunk
+      (lambda ()
+       (set! module-defer-observers #f)
+       (hash-for-each (lambda (m dummy)
+                        (module-call-observers m))
+                      module-defer-observers-table)
+       (hash-clear! module-defer-observers-table)
+       (unlock-mutex module-defer-observers-mutex))))
+
+(define (module-call-observers m)
   (for-each (lambda (proc) (proc m)) (module-observers m))
   (hash-fold (lambda (id proc res) (proc m)) #f (module-weak-observers m)))
 
 \f
+
 ;;; {Module Searching in General}
 ;;;
 ;;; We sometimes want to look for properties of a symbol
 ;;
 (define (module-symbol-local-binding m v . opt-val)
   (let ((var (module-local-variable m v)))
-    (if var
+    (if (and var (variable-bound? var))
        (variable-ref var)
        (if (not (null? opt-val))
            (car opt-val)
 ;;
 (define (module-symbol-binding m v . opt-val)
   (let ((var (module-variable m v)))
-    (if var
+    (if (and var (variable-bound? var))
        (variable-ref var)
        (if (not (null? opt-val))
            (car opt-val)
 
 
 \f
+
 ;;; {Adding Variables to Modules}
 ;;;
-;;;
-
 
 ;; module-make-local-var! module symbol
 ;;
 ;; If no variable was already there, then create a new and uninitialzied
 ;; variable.
 ;;
+;; This function is used in modules.c.
+;;
 (define (module-make-local-var! m v)
   (or (let ((b (module-obarray-ref (module-obarray m) v)))
        (and (variable? b)
             (begin
+              ;; Mark as modified since this function is called when
+              ;; the standard eval closure defines a binding
               (module-modified m)
               b)))
-      (and (module-binder m)
-          ((module-binder m) m v #t))
-      (begin
-       (let ((answer (make-undefined-variable)))
-         (module-obarray-set! (module-obarray m) v answer)
-         (module-modified m)
-         answer))))
+
+      ;; No local variable yet, so we need to create a new one.  That
+      ;; new variable is initialized with the old imported value of V,
+      ;; if there is one.
+      (let ((imported-var (module-variable m v))
+           (local-var (or (and (module-binder m)
+                               ((module-binder m) m v #t))
+                          (begin
+                            (let ((answer (make-undefined-variable)))
+                              (module-add! m v answer)
+                              answer)))))
+       (if (and imported-var (not (variable-bound? local-var)))
+           (variable-set! local-var (variable-ref imported-var)))
+       local-var)))
 
 ;; module-ensure-local-variable! module symbol
 ;;
 ;; make sure that a symbol is undefined in the local namespace of M.
 ;;
 (define (module-remove! m v)
-  (module-obarray-remove!  (module-obarray m) v)
+  (module-obarray-remove! (module-obarray m) v)
   (module-modified m))
 
 (define (module-clear! m)
-  (vector-fill! (module-obarray m) '())
+  (hash-clear! (module-obarray m))
   (module-modified m))
 
 ;; MODULE-FOR-EACH -- exported
 ;; Call PROC on each symbol in MODULE, with arguments of (SYMBOL VARIABLE).
 ;;
 (define (module-for-each proc module)
-  (let ((obarray (module-obarray module)))
-    (do ((index 0 (+ index 1))
-        (end (vector-length obarray)))
-       ((= index end))
-      (for-each
-       (lambda (bucket)
-        (proc (car bucket) (cdr bucket)))
-       (vector-ref obarray index)))))
-
+  (hash-for-each proc (module-obarray module)))
 
 (define (module-map proc module)
-  (let* ((obarray (module-obarray module))
-        (end (vector-length obarray)))
-
-    (let loop ((i 0)
-              (answer '()))
-      (if (= i end)
-         answer
-         (loop (+ 1 i)
-               (append!
-                (map (lambda (bucket)
-                       (proc (car bucket) (cdr bucket)))
-                     (vector-ref obarray i))
-                answer))))))
+  (hash-map->list proc (module-obarray module)))
+
 \f
 
 ;;; {Low Level Bootstrapping}
 
 
 \f
+
 ;;; {Module-based Loading}
 ;;;
 
 
 (define basic-load load)
 
-(define (load-module filename)
+(define (load-module filename . reader)
   (save-module-excursion
    (lambda ()
      (let ((oldname (and (current-load-port)
                         (port-filename (current-load-port)))))
-       (basic-load (if (and oldname
-                           (> (string-length filename) 0)
-                           (not (char=? (string-ref filename 0) #\/))
-                           (not (string=? (dirname oldname) ".")))
-                      (string-append (dirname oldname) "/" filename)
-                      filename))))))
+       (apply basic-load
+             (if (and oldname
+                      (> (string-length filename) 0)
+                      (not (char=? (string-ref filename 0) #\/))
+                      (not (string=? (dirname oldname) ".")))
+                 (string-append (dirname oldname) "/" filename)
+                 filename)
+             reader)))))
 
 
 \f
+
 ;;; {MODULE-REF -- exported}
-;;
+;;;
+
 ;; Returns the value of a variable called NAME in MODULE or any of its
 ;; used modules.  If there is no such variable, then if the optional third
 ;; argument DEFAULT is present, it is returned; otherwise an error is signaled.
 ;;
 (define (module-use! module interface)
   (set-module-uses! module
-                   (cons interface (delq! interface (module-uses module))))
+                   (cons interface
+                         (filter (lambda (m)
+                                   (not (equal? (module-name m)
+                                                (module-name interface))))
+                                 (module-uses module))))
   (module-modified module))
 
+;; MODULE-USE-INTERFACES! module interfaces
+;;
+;; Same as MODULE-USE! but add multiple interfaces and check for duplicates
+;;
+(define (module-use-interfaces! module interfaces)
+  (let* ((duplicates-handlers? (or (module-duplicates-handlers module)
+                                  (default-duplicate-binding-procedures)))
+        (uses (module-uses module)))
+    ;; remove duplicates-interface
+    (set! uses (delq! (module-duplicates-interface module) uses))
+    ;; remove interfaces to be added
+    (for-each (lambda (interface)
+               (set! uses
+                     (filter (lambda (m)
+                               (not (equal? (module-name m)
+                                            (module-name interface))))
+                             uses)))
+             interfaces)
+    ;; add interfaces to use list
+    (set-module-uses! module uses)
+    (for-each (lambda (interface)
+               (and duplicates-handlers?
+                    ;; perform duplicate checking
+                    (process-duplicates module interface))
+               (set! uses (cons interface uses))
+               (set-module-uses! module uses))
+             interfaces)
+    ;; add duplicates interface
+    (if (module-duplicates-interface module)
+       (set-module-uses! module
+                         (cons (module-duplicates-interface module) uses)))
+    (module-modified module)))
+
 \f
+
 ;;; {Recursive Namespaces}
 ;;;
-;;;
 ;;; A hierarchical namespace emerges if we consider some module to be
 ;;; root, and variables bound to modules as nested namespaces.
 ;;;
 
 
 \f
-;;; {The (app) module}
+
+;;; {The (%app) module}
 ;;;
 ;;; The root of conventionally named objects not directly in the top level.
 ;;;
-;;; (app modules)
-;;; (app modules guile)
+;;; (%app modules)
+;;; (%app modules guile)
 ;;;
 ;;; The directory of all modules and the standard root module.
 ;;;
          (set-module-public-interface! module interface))))
   (if (and (not (memq the-scm-module (module-uses module)))
           (not (eq? module the-root-module)))
-      (set-module-uses! module (append (module-uses module) (list the-scm-module)))))
+      (set-module-uses! module
+                       (append (module-uses module) (list the-scm-module)))))
 
 ;; NOTE: This binding is used in libguile/modules.c.
 ;;
 (define (resolve-module name . maybe-autoload)
-  (let ((full-name (append '(app modules) name)))
-    (let ((already (local-ref full-name)))
+  (let ((full-name (append '(%app modules) name)))
+    (let ((already (nested-ref the-root-module full-name)))
       (if already
          ;; The module already exists...
          (if (and (or (null? maybe-autoload) (car maybe-autoload))
 ;;
 (set-current-module the-root-module)
 
-(define app (make-module 31))
-(local-define '(app modules) (make-module 31))
-(local-define '(app modules guile) the-root-module)
+(define %app (make-module 31))
+(define app %app) ;; for backwards compatability
+(local-define '(%app modules) (make-module 31))
+(local-define '(%app modules guile) the-root-module)
 
-;; (define-special-value '(app modules new-ws) (lambda () (make-scm-module)))
+;; (define-special-value '(%app modules new-ws) (lambda () (make-scm-module)))
 
 (define (try-load-module name)
-  (try-module-autoload name))
+  (or (begin-deprecated (try-module-linked name))
+      (try-module-autoload name)
+      (begin-deprecated (try-module-dynamic-link name))))
 
 (define (purify-module! module)
   "Removes bindings in MODULE which are inherited from the (guile) module."
 ;; Return a module that is an interface to the module designated by
 ;; NAME.
 ;;
-;; `resolve-interface' takes two keyword arguments:
+;; `resolve-interface' takes four keyword arguments:
 ;;
 ;;   #:select SELECTION
 ;;
 ;; are made available in the interface.  Bindings that are added later
 ;; are not picked up.
 ;;
+;;   #:hide BINDINGS
+;;
+;; BINDINGS is a list of bindings which should not be imported.
+;;
+;;   #:prefix PREFIX
+;;
+;; PREFIX is a symbol that will be appended to each exported name.
+;; The default is to not perform any renaming.
+;;
 ;;   #:renamer RENAMER
 ;;
 ;; RENAMER is a procedure that takes a symbol and returns its new
-;; name.  The default is to not perform any renaming.
+;; name.  The default is not perform any renaming.
 ;;
 ;; Signal "no code for module" error if module name is not resolvable
 ;; or its public interface is not available.  Signal "no binding"
           def)))
 
   (let* ((select (get-keyword-arg args #:select #f))
-        (renamer (get-keyword-arg args #:renamer identity))
+        (hide (get-keyword-arg args #:hide '()))
+        (renamer (or (get-keyword-arg args #:renamer #f)
+                     (let ((prefix (get-keyword-arg args #:prefix #f)))
+                       (and prefix (symbol-prefix-proc prefix)))
+                     identity))
          (module (resolve-module name))
          (public-i (and module (module-public-interface module))))
     (and (or (not module) (not public-i))
          (error "no code for module" name))
-    (if (and (not select) (eq? renamer identity))
+    (if (and (not select) (null? hide) (eq? renamer identity))
         public-i
         (let ((selection (or select (module-map (lambda (sym var) sym)
                                                public-i)))
               (custom-i (make-module 31)))
-          (set-module-kind! custom-i 'interface)
+          (set-module-kind! custom-i 'custom-interface)
+         (set-module-name! custom-i name)
          ;; XXX - should use a lazy binder so that changes to the
          ;; used module are picked up automatically.
-          (for-each (lambda (bspec)
-                      (let* ((direct? (symbol? bspec))
-                             (orig (if direct? bspec (car bspec)))
-                             (seen (if direct? bspec (cdr bspec))))
-                        (module-add! custom-i (renamer seen)
-                                     (or (module-local-variable public-i orig)
-                                         (module-local-variable module orig)
-                                         (error
-                                          ;; fixme: format manually for now
-                                          (simple-format
-                                           #f "no binding `~A' in module ~A"
-                                           orig name))))))
-                    selection)
+         (for-each (lambda (bspec)
+                     (let* ((direct? (symbol? bspec))
+                            (orig (if direct? bspec (car bspec)))
+                            (seen (if direct? bspec (cdr bspec)))
+                            (var (or (module-local-variable public-i orig)
+                                     (module-local-variable module orig)
+                                     (error
+                                      ;; fixme: format manually for now
+                                      (simple-format
+                                       #f "no binding `~A' in module ~A"
+                                       orig name)))))
+                       (if (memq orig hide)
+                           (set! hide (delq! orig hide))
+                           (module-add! custom-i
+                                        (renamer seen)
+                                        var))))
+                   selection)
+         ;; Check that we are not hiding bindings which don't exist
+         (for-each (lambda (binding)
+                     (if (not (module-local-variable public-i binding))
+                         (error
+                          (simple-format
+                           #f "no binding `~A' to hide in module ~A"
+                           binding name))))
+                   hide)
           custom-i))))
 
 (define (symbol-prefix-proc prefix)
     (let loop ((kws kws)
               (reversed-interfaces '())
               (exports '())
-              (re-exports '()))
+              (re-exports '())
+              (replacements '()))
+
       (if (null? kws)
-         (begin
-           (for-each (lambda (interface)
-                       (module-use! module interface))
-                     (reverse reversed-interfaces))
-           (module-export! module exports)
-           (module-re-export! module re-exports))
+         (call-with-deferred-observers
+          (lambda ()
+            (module-use-interfaces! module (reverse reversed-interfaces))
+            (module-export! module exports)
+            (module-replace! module replacements)
+            (module-re-export! module re-exports)))
          (case (car kws)
            ((#:use-module #:use-syntax)
             (or (pair? (cdr kws))
               (loop (cddr kws)
                     (cons interface reversed-interfaces)
                     exports
-                    re-exports)))
+                    re-exports
+                    replacements)))
            ((#:autoload)
             (or (and (pair? (cdr kws)) (pair? (cddr kws)))
                 (unrecognized kws))
                                                  (caddr kws))
                         reversed-interfaces)
                   exports
-                  re-exports))
+                  re-exports
+                  replacements))
            ((#:no-backtrace)
             (set-system-module! module #t)
-            (loop (cdr kws) reversed-interfaces exports re-exports))
+            (loop (cdr kws) reversed-interfaces exports re-exports replacements))
            ((#:pure)
             (purify-module! module)
-            (loop (cdr kws) reversed-interfaces exports re-exports))
+            (loop (cdr kws) reversed-interfaces exports re-exports replacements))
+           ((#:duplicates)
+            (if (not (pair? (cdr kws)))
+                (unrecognized kws))
+            (set-module-duplicates-handlers!
+             module
+             (lookup-duplicates-handlers (cadr kws)))
+            (loop (cddr kws) reversed-interfaces exports re-exports replacements))
            ((#:export #:export-syntax)
             (or (pair? (cdr kws))
                 (unrecognized kws))
             (loop (cddr kws)
                   reversed-interfaces
                   (append (cadr kws) exports)
-                  re-exports))
+                  re-exports
+                  replacements))
            ((#:re-export #:re-export-syntax)
             (or (pair? (cdr kws))
                 (unrecognized kws))
             (loop (cddr kws)
                   reversed-interfaces
                   exports
-                  (append (cadr kws) re-exports)))
+                  (append (cadr kws) re-exports)
+                  replacements))
+           ((#:replace #:replace-syntax)
+            (or (pair? (cdr kws))
+                (unrecognized kws))
+            (loop (cddr kws)
+                  reversed-interfaces
+                  exports
+                  re-exports
+                  (append (cadr kws) replacements)))
            (else
             (unrecognized kws)))))
+    (run-hook module-defined-hook module)
     module))
 
+;; `module-defined-hook' is a hook that is run whenever a new module
+;; is defined.  Its members are called with one argument, the new
+;; module.
+(define module-defined-hook (make-hook 1))
+
+\f
+
 ;;; {Autoload}
+;;;
 
 (define (make-autoload-interface module name bindings)
   (let ((b (lambda (a sym definep)
                  (let ((i (module-public-interface (resolve-module name))))
                    (if (not i)
                        (error "missing interface for module" name))
-                   ;; Replace autoload-interface with interface
-                   (set-car! (memq a (module-uses module)) i)
+                   (let ((autoload (memq a (module-uses module))))
+                     ;; Replace autoload-interface with actual interface if
+                     ;; that has not happened yet.
+                     (if (pair? autoload)
+                         (set-car! autoload i)))
                    (module-local-variable i sym))))))
-    (module-constructor #() '() b #f #f name 'autoload
+    (module-constructor (make-hash-table 0) '() b #f #f name 'autoload #f #f
                        '() (make-weak-value-hash-table 31) 0)))
 
 ;;; {Compiled module}
 (define load-compiled #f)
 
 \f
+
 ;;; {Autoloading modules}
+;;;
 
 (define autoloads-in-progress '())
 
                            (load-file load-compiled full)))
                      ((%search-load-path file)
                       => (lambda (full)
-                           (load-file primitive-load full))))))
+                           (with-fluids ((current-reader #f))
+                             (load-file primitive-load full)))))))
            (lambda () (set-autoloaded! dir-hint name didit)))
           didit))))
 
 \f
-;;; Dynamic linking of modules
+
+;;; {Dynamic linking of modules}
+;;;
 
 (define autoloads-done '((guile . guile)))
 
        (set! autoloads-done (delete! n autoloads-done))
        (set! autoloads-in-progress (delete! n autoloads-in-progress)))))
 
-
-
-\f
-;; {EVAL-CASE}
-;;
-;; (eval-case ((situation*) forms)* (else forms)?)
-;;
-;; Evaluate certain code based on the situation that eval-case is used
-;; in.  The only defined situation right now is `load-toplevel' which
-;; triggers for code evaluated at the top-level, for example from the
-;; REPL or when loading a file.
-
-(define eval-case
-  (procedure->memoizing-macro
-   (lambda (exp env)
-     (define (toplevel-env? env)
-       (or (not (pair? env)) (not (pair? (car env)))))
-     (define (syntax)
-       (error "syntax error in eval-case"))
-     (let loop ((clauses (cdr exp)))
-       (cond
-       ((null? clauses)
-        #f)
-       ((not (list? (car clauses)))
-        (syntax))
-       ((eq? 'else (caar clauses))
-        (or (null? (cdr clauses))
-            (syntax))
-        (cons 'begin (cdar clauses)))
-       ((not (list? (caar clauses)))
-        (syntax))
-       ((and (toplevel-env? env)
-             (memq 'load-toplevel (caar clauses)))
-        (cons 'begin (cdar clauses)))
-       (else
-        (loop (cdr clauses))))))))
-
-\f
-;;; {Macros}
-;;;
-
-(define (primitive-macro? m)
-  (and (macro? m)
-       (not (macro-transformer m))))
-
-;;; {Defmacros}
-;;;
-(define macro-table (make-weak-key-hash-table 523))
-(define xformer-table (make-weak-key-hash-table 523))
-
-(define (defmacro? m)  (hashq-ref macro-table m))
-(define (assert-defmacro?! m) (hashq-set! macro-table m #t))
-(define (defmacro-transformer m) (hashq-ref xformer-table m))
-(define (set-defmacro-transformer! m t) (hashq-set! xformer-table m t))
-
-(define defmacro:transformer
-  (lambda (f)
-    (let* ((xform (lambda (exp env)
-                   (copy-tree (apply f (cdr exp)))))
-          (a (procedure->memoizing-macro xform)))
-      (assert-defmacro?! a)
-      (set-defmacro-transformer! a f)
-      a)))
-
-
-(define defmacro
-  (let ((defmacro-transformer
-         (lambda (name parms . body)
-           (let ((transformer `(lambda ,parms ,@body)))
-             `(eval-case
-               ((load-toplevel)
-                (define ,name (defmacro:transformer ,transformer)))
-               (else
-                (error "defmacro can only be used at the top level")))))))
-    (defmacro:transformer defmacro-transformer)))
-
-(define defmacro:syntax-transformer
-  (lambda (f)
-    (procedure->syntax
-             (lambda (exp env)
-               (copy-tree (apply f (cdr exp)))))))
-
-
-;; XXX - should the definition of the car really be looked up in the
-;; current module?
-
-(define (macroexpand-1 e)
-  (cond
-   ((pair? e) (let* ((a (car e))
-                    (val (and (symbol? a) (local-ref (list a)))))
-               (if (defmacro? val)
-                   (apply (defmacro-transformer val) (cdr e))
-                   e)))
-   (#t e)))
-
-(define (macroexpand e)
-  (cond
-   ((pair? e) (let* ((a (car e))
-                    (val (and (symbol? a) (local-ref (list a)))))
-               (if (defmacro? val)
-                   (macroexpand (apply (defmacro-transformer val) (cdr e)))
-                   e)))
-   (#t e)))
-
-(provide 'defmacro)
-
 \f
 
 ;;; {Run-time options}
+;;;
 
 (define define-option-interface
   (let* ((option-name car)
                                          (set! options (delq! flag options)))
                                        flags)
                              (,interface options)
-                             (,interface)))))
-
-        (make-set! (lambda (interface)
-                     `((name exp)
-                       (,'quasiquote
-                        (begin (,interface (append (,interface)
-                                                   (list '(,'unquote name)
-                                                         (,'unquote exp))))
-                               (,interface)))))))
+                             (,interface))))))
     (procedure->memoizing-macro
      (lambda (exp env)
-       (cons 'begin
-            (let* ((option-group (cadr exp))
-                   (interface (car option-group)))
-              (append (map (lambda (name constructor)
-                             `(define ,name
-                                ,(constructor interface)))
-                           (cadr option-group)
-                           (list make-options
-                                 make-enable
-                                 make-disable))
-                      (map (lambda (name constructor)
-                             `(defmacro ,name
-                                ,@(constructor interface)))
-                           (caddr option-group)
-                           (list make-set!)))))))))
+       (let* ((option-group (cadr exp))
+             (interface (car option-group))
+             (options/enable/disable (cadr option-group)))
+        `(begin
+           (define ,(car options/enable/disable)
+             ,(make-options interface))
+           (define ,(cadr options/enable/disable)
+             ,(make-enable interface))
+           (define ,(caddr options/enable/disable)
+             ,(make-disable interface))
+           (defmacro ,(caaddr option-group) (opt val)
+             `(,,(car options/enable/disable)
+               (append (,,(car options/enable/disable))
+                       (list ',opt ,val))))))))))
 
 (define-option-interface
   (eval-options-interface
             (catch #t
 
                    (lambda ()
-                     (lazy-catch #t
-                                 (lambda ()
-                                   (dynamic-wind
-                                    (lambda () (unmask-signals))
-                                    (lambda ()
-                                      (with-traps
-                                       (lambda ()
-                                         (first)
-
-                                         ;; This line is needed because mark
-                                         ;; doesn't do closures quite right.
-                                         ;; Unreferenced locals should be
-                                         ;; collected.
-                                         ;;
-                                         (set! first #f)
-                                         (let loop ((v (thunk)))
-                                           (loop (thunk)))
-                                         #f)))
-                                    (lambda () (mask-signals))))
-
-                                 lazy-handler-dispatch))
+                     (call-with-unblocked-asyncs
+                      (lambda ()
+                        (with-traps
+                         (lambda ()
+                           (first)
+
+                           ;; This line is needed because mark
+                           ;; doesn't do closures quite right.
+                           ;; Unreferenced locals should be
+                           ;; collected.
+                           (set! first #f)
+                           (let loop ((v (thunk)))
+                             (loop (thunk)))
+                           #f)))))
 
                    (lambda (key . args)
                      (case key
                           (cond ((= (length args) 4)
                                  (apply handle-system-error key args))
                                 (else
-                                 (apply bad-throw key args))))))))))
+                                 (apply bad-throw key args)))))))
+
+                   ;; Note that having just `lazy-handler-dispatch'
+                   ;; here is connected with the mechanism that
+                   ;; produces a nice backtrace upon error.  If, for
+                   ;; example, this is replaced with (lambda args
+                   ;; (apply lazy-handler-dispatch args)), the stack
+                   ;; cutting (in save-stack) goes wrong and ends up
+                   ;; saving no stack at all, so there is no
+                   ;; backtrace.
+                   lazy-handler-dispatch)))
+
        (if next (loop next) status)))
     (set! set-batch-mode?! (lambda (arg)
                             (cond (arg
                                   (#t
                                    (error "sorry, not implemented")))))
     (set! batch-mode? (lambda () (not interactive)))
-    (loop (lambda () #t))))
+    (call-with-blocked-asyncs
+     (lambda () (loop (lambda () #t))))))
 
 ;;(define the-last-stack (make-fluid)) Defined by scm_init_backtrace ()
 (define before-signal-stack (make-fluid))
   (let ((cep (current-error-port)))
     (cond ((not (stack? (fluid-ref the-last-stack))))
          ((memq 'backtrace (debug-options-interface))
-          (run-hook before-backtrace-hook)
-          (newline cep)
-          (display "Backtrace:\n")
-          (display-backtrace (fluid-ref the-last-stack) cep)
-          (newline cep)
-          (run-hook after-backtrace-hook)))
+          (let ((highlights (if (or (eq? key 'wrong-type-arg)
+                                    (eq? key 'out-of-range))
+                                (list-ref args 3)
+                                '())))
+            (run-hook before-backtrace-hook)
+            (newline cep)
+            (display "Backtrace:\n")
+            (display-backtrace (fluid-ref the-last-stack) cep
+                               #f #f highlights)
+            (newline cep)
+            (run-hook after-backtrace-hook))))
     (run-hook before-error-hook)
     (apply display-error (fluid-ref the-last-stack) cep args)
     (run-hook after-error-hook)
     (display prompt)
     (force-output)
     (run-hook before-read-hook)
-    (read (current-input-port))))
+    ((or (fluid-ref current-reader) read) (current-input-port))))
 
 (define (scm-style-repl)
 
 
 
 \f
+
 ;;; {IOTA functions: generating lists of numbers}
+;;;
 
 (define (iota n)
   (let loop ((count (1- n)) (result '()))
         (loop (1- count) (cons count result)))))
 
 \f
-;;; {While}
-;;;
-;;; with `continue' and `break'.
-;;;
-
-(defmacro while (cond . body)
-  `(letrec ((continue (lambda () (or (not ,cond) (begin (begin ,@ body) (continue)))))
-           (break (lambda val (apply throw 'break val))))
-     (catch 'break
-           (lambda () (continue))
-           (lambda v (cadr v)))))
 
 ;;; {collect}
 ;;;
 ;;; forms instead of the result of the last form.
 ;;; (The definition relies on the current left-to-right
 ;;;  order of evaluation of operands in applications.)
+;;;
 
 (defmacro collect forms
   (cons 'list forms))
 
+\f
+
 ;;; {with-fluids}
+;;;
 
 ;; with-fluids is a convenience wrapper for the builtin procedure
 ;; `with-fluids*'.  The syntax is just like `let':
 ;;     body)
 
 (defmacro with-fluids (bindings . body)
-  `(with-fluids* (list ,@(map car bindings)) (list ,@(map cadr bindings))
-                (lambda () ,@body)))
+  (let ((fluids (map car bindings))
+       (values (map cadr bindings)))
+    (if (and (= (length fluids) 1) (= (length values) 1))
+       `(with-fluid* ,(car fluids) ,(car values) (lambda () ,@body))
+       `(with-fluids* (list ,@fluids) (list ,@values)
+                      (lambda () ,@body)))))
 
 \f
 
 ;; coaxing
 ;;
 
+(define (primitive-macro? m)
+  (and (macro? m)
+       (not (macro-transformer m))))
+
 (defmacro define-macro (first . rest)
   (let ((name (if (symbol? first) first (car first)))
        (transformer
        (error "define-syntax-macro can only be used at the top level")))))
 
 \f
+
+;;; {While}
+;;;
+;;; with `continue' and `break'.
+;;;
+
+;; The inner `do' loop avoids re-establishing a catch every iteration,
+;; that's only necessary if continue is actually used.  A new key is
+;; generated every time, so break and continue apply to their originating
+;; `while' even when recursing.  `while-helper' is an easy way to keep the
+;; `key' binding away from the cond and body code.
+;;
+;; FIXME: This is supposed to have an `unquote' on the `do' the same used
+;; for lambda and not, so as to protect against any user rebinding of that
+;; symbol, but unfortunately an unquote breaks with ice-9 syncase, eg.
+;;
+;;     (use-modules (ice-9 syncase))
+;;     (while #f)
+;;     => ERROR: invalid syntax ()
+;;
+;; This is probably a bug in syncase.
+;;
+(define-macro (while cond . body)
+  (define (while-helper proc)
+    (do ((key (make-symbol "while-key")))
+       ((catch key
+               (lambda ()
+                 (proc (lambda () (throw key #t))
+                       (lambda () (throw key #f))))
+               (lambda (key arg) arg)))))
+  `(,while-helper (,lambda (break continue)
+                   (do ()
+                       ((,not ,cond))
+                     ,@body)
+                   #t)))
+
+
+\f
+
 ;;; {Module System Macros}
 ;;;
 
   (define keys
     ;; sym     key      quote?
     '((:select #:select #t)
+      (:hide   #:hide  #t)
+      (:prefix #:prefix #t)
       (:renamer #:renamer #f)))
   (if (not (pair? (car spec)))
       `(',spec)
 ;; to change scm_c_use_module as well.
 
 (define (process-use-modules module-interface-args)
-  (for-each (lambda (mif-args)
-             (let ((mod-iface (apply resolve-interface mif-args)))
-               (or mod-iface
-                   (error "no such module" mif-args))
-               (module-use! (current-module) mod-iface)))
-           module-interface-args))
+  (let ((interfaces (map (lambda (mif-args)
+                          (or (apply resolve-interface mif-args)
+                              (error "no such module" mif-args)))
+                        module-interface-args)))
+    (call-with-deferred-observers
+     (lambda ()
+       (module-use-interfaces! (current-module) interfaces)))))
 
 (defmacro use-modules modules
   `(eval-case
      (process-use-modules
       (list ,@(map (lambda (m)
                     `(list ,@(compile-interface-spec m)))
-                  modules))))
+                  modules)))
+     *unspecified*)
     (else
      (error "use-modules can only be used at the top level"))))
 
                                   (list ,@(compile-interface-spec spec))))
             (set-module-transformer! (current-module)
                                      ,(car (last-pair spec))))
-          `((set-module-transformer! (current-module) ,spec))))
+          `((set-module-transformer! (current-module) ,spec)))
+     *unspecified*)
     (else
      (error "use-syntax can only be used at the top level"))))
 
+;; Dirk:FIXME:: This incorrect (according to R5RS) syntax needs to be changed
+;; as soon as guile supports hygienic macros.
 (define define-private define)
 
 (defmacro define-public args
    (#t
     (let ((name (defined-name (car args))))
       `(begin
-        (eval-case ((load-toplevel) (export ,name)))
+        (eval-case ((load-toplevel) (export-syntax ,name)))
         (defmacro ,@args))))))
 
 ;; Export a local variable
                  (module-add! public-i name var)))
              names)))
 
+(define (module-replace! m names)
+  (let ((public-i (module-public-interface m)))
+    (for-each (lambda (name)
+               (let ((var (module-ensure-local-variable! m name)))
+                 (set-object-property! var 'replace #t)
+                 (module-add! public-i name var)))
+             names)))
+
 ;; Re-export a imported variable
 ;;
 (define (module-re-export! m names)
 (defmacro export names
   `(eval-case
     ((load-toplevel)
-     (module-export! (current-module) ',names))
+     (call-with-deferred-observers
+      (lambda ()
+       (module-export! (current-module) ',names))))
     (else
      (error "export can only be used at the top level"))))
 
 (defmacro re-export names
   `(eval-case
     ((load-toplevel)
-     (module-re-export! (current-module) ',names))
+     (call-with-deferred-observers
+      (lambda ()
+       (module-re-export! (current-module) ',names))))
     (else
      (error "re-export can only be used at the top level"))))
 
-(define export-syntax export)
-(define re-export-syntax re-export)
+(defmacro export-syntax names
+  `(export ,@names))
 
+(defmacro re-export-syntax names
+  `(re-export ,@names))
 
 (define load load-module)
 
+;; The following macro allows one to write, for example,
+;;
+;;    (@ (ice-9 pretty-print) pretty-print)
+;;
+;; to refer directly to the pretty-print variable in module (ice-9
+;; pretty-print).  It works by looking up the variable and inserting
+;; it directly into the code.  This is understood by the evaluator.
+;; Indeed, all references to global variables are memoized into such
+;; variable objects.
+
+(define-macro (@ mod-name var-name)
+  (let ((var (module-variable (resolve-interface mod-name) var-name)))
+    (if (not var)
+       (error "no such public variable" (list '@ mod-name var-name)))
+    var))
+
+;; The '@@' macro is like '@' but it can also access bindings that
+;; have not been explicitely exported.
+
+(define-macro (@@ mod-name var-name)
+  (let ((var (module-variable (resolve-module mod-name) var-name)))
+    (if (not var)
+       (error "no such variable" (list '@@ mod-name var-name)))
+    var))
+
+\f
+
+;;; {Parameters}
+;;;
+
+(define make-mutable-parameter
+  (let ((make (lambda (fluid converter)
+               (lambda args
+                 (if (null? args)
+                     (fluid-ref fluid)
+                     (fluid-set! fluid (converter (car args))))))))
+    (lambda (init . converter)
+      (let ((fluid (make-fluid))
+           (converter (if (null? converter)
+                          identity
+                          (car converter))))
+       (fluid-set! fluid (converter init))
+       (make fluid converter)))))
+
+\f
+
+;;; {Handling of duplicate imported bindings}
+;;;
+
+;; Duplicate handlers take the following arguments:
+;;
+;; module  importing module
+;; name           conflicting name
+;; int1           old interface where name occurs
+;; val1           value of binding in old interface
+;; int2           new interface where name occurs
+;; val2           value of binding in new interface
+;; var    previous resolution or #f
+;; val    value of previous resolution
+;;
+;; A duplicate handler can take three alternative actions:
+;;
+;; 1. return #f => leave responsibility to next handler
+;; 2. exit with an error
+;; 3. return a variable resolving the conflict
+;;
+
+(define duplicate-handlers
+  (let ((m (make-module 7)))
+    
+    (define (check module name int1 val1 int2 val2 var val)
+      (scm-error 'misc-error
+                #f
+                "~A: `~A' imported from both ~A and ~A"
+                (list (module-name module)
+                      name
+                      (module-name int1)
+                      (module-name int2))
+                #f))
+    
+    (define (warn module name int1 val1 int2 val2 var val)
+      (format #t
+             "WARNING: ~A: `~A' imported from both ~A and ~A\n"
+             (module-name module)
+             name
+             (module-name int1)
+             (module-name int2))
+      #f)
+     
+    (define (replace module name int1 val1 int2 val2 var val)
+      (let ((old (or (and var (object-property var 'replace) var)
+                    (module-variable int1 name)))
+           (new (module-variable int2 name)))
+       (if (object-property old 'replace)
+           (and (or (eq? old new)
+                    (not (object-property new 'replace)))
+                old)
+           (and (object-property new 'replace)
+                new))))
+    
+    (define (warn-override-core module name int1 val1 int2 val2 var val)
+      (and (eq? int1 the-scm-module)
+          (begin
+            (format #t
+                    "WARNING: ~A: imported module ~A overrides core binding `~A'\n"
+                    (module-name module)
+                    (module-name int2)
+                    name)
+            (module-local-variable int2 name))))
+     
+    (define (first module name int1 val1 int2 val2 var val)
+      (or var (module-local-variable int1 name)))
+     
+    (define (last module name int1 val1 int2 val2 var val)
+      (module-local-variable int2 name))
+     
+    (define (noop module name int1 val1 int2 val2 var val)
+      #f)
+    
+    (set-module-name! m 'duplicate-handlers)
+    (set-module-kind! m 'interface)
+    (module-define! m 'check check)
+    (module-define! m 'warn warn)
+    (module-define! m 'replace replace)
+    (module-define! m 'warn-override-core warn-override-core)
+    (module-define! m 'first first)
+    (module-define! m 'last last)
+    (module-define! m 'merge-generics noop)
+    (module-define! m 'merge-accessors noop)
+    m))
+
+(define (lookup-duplicates-handlers handler-names)
+  (and handler-names
+       (map (lambda (handler-name)
+             (or (module-symbol-local-binding
+                  duplicate-handlers handler-name #f)
+                 (error "invalid duplicate handler name:"
+                        handler-name)))
+           (if (list? handler-names)
+               handler-names
+               (list handler-names)))))
+
+(define default-duplicate-binding-procedures
+  (make-mutable-parameter #f))
+
+(define default-duplicate-binding-handler
+  (make-mutable-parameter '(replace warn-override-core warn last)
+                         (lambda (handler-names)
+                           (default-duplicate-binding-procedures
+                             (lookup-duplicates-handlers handler-names))
+                           handler-names)))
+
+(define (make-duplicates-interface)
+  (let ((m (make-module)))
+    (set-module-kind! m 'custom-interface)
+    (set-module-name! m 'duplicates)
+    m))
+
+(define (process-duplicates module interface)
+  (let* ((duplicates-handlers (or (module-duplicates-handlers module)
+                                 (default-duplicate-binding-procedures)))
+        (duplicates-interface (module-duplicates-interface module)))
+    (module-for-each
+     (lambda (name var)
+       (cond ((module-import-interface module name)
+             =>
+             (lambda (prev-interface)
+               (let ((var1 (module-local-variable prev-interface name))
+                     (var2 (module-local-variable interface name)))
+                 (if (not (eq? var1 var2))
+                     (begin
+                       (if (not duplicates-interface)
+                           (begin
+                             (set! duplicates-interface
+                                   (make-duplicates-interface))
+                             (set-module-duplicates-interface!
+                              module
+                              duplicates-interface)))
+                       (let* ((var (module-local-variable duplicates-interface
+                                                          name))
+                              (val (and var
+                                        (variable-bound? var)
+                                        (variable-ref var))))
+                         (let loop ((duplicates-handlers duplicates-handlers))
+                           (cond ((null? duplicates-handlers))
+                                 (((car duplicates-handlers)
+                                   module
+                                   name
+                                   prev-interface
+                                   (and (variable-bound? var1)
+                                        (variable-ref var1))
+                                   interface
+                                   (and (variable-bound? var2)
+                                        (variable-ref var2))
+                                   var
+                                   val)
+                                  =>
+                                  (lambda (var)
+                                    (module-add! duplicates-interface name var)))
+                                 (else
+                                  (loop (cdr duplicates-handlers)))))))))))))
+     interface)))
+
 \f
 
 ;;; {`cond-expand' for SRFI-0 support.}
 ;;;
 ;;; Currently, the following feature identifiers are supported:
 ;;;
-;;;   guile r5rs srfi-0
+;;;   guile r5rs srfi-0 srfi-4 srfi-6 srfi-13 srfi-14 srfi-55 srfi-61
 ;;;
 ;;; Remember to update the features list when adding more SRFIs.
+;;;
 
 (define %cond-expand-features
   ;; Adjust the above comment when changing this.
-  '(guile r5rs srfi-0))
+  '(guile
+    r5rs
+    srfi-0   ;; cond-expand itself
+    srfi-4   ;; homogenous numeric vectors
+    srfi-6   ;; open-input-string etc, in the guile core
+    srfi-13  ;; string library
+    srfi-14  ;; character sets
+    srfi-55  ;; require-extension
+    srfi-61  ;; general cond clause
+    ))
 
 ;; This table maps module public interfaces to the list of features.
 ;;
 
 \f
 
+;;; srfi-55: require-extension
+;;;
+
+(define-macro (require-extension extension-spec)
+  ;; This macro only handles the srfi extension, which, at present, is
+  ;; the only one defined by the standard.
+  (if (not (pair? extension-spec))
+      (scm-error 'wrong-type-arg "require-extension"
+                 "Not an extension: ~S" (list extension-spec) #f))
+  (let ((extension (car extension-spec))
+        (extension-args (cdr extension-spec)))
+    (case extension
+      ((srfi)
+       (let ((use-list '()))
+         (for-each
+          (lambda (i)
+            (if (not (integer? i))
+                (scm-error 'wrong-type-arg "require-extension"
+                           "Invalid srfi name: ~S" (list i) #f))
+            (let ((srfi-sym (string->symbol
+                             (string-append "srfi-" (number->string i)))))
+              (if (not (memq srfi-sym %cond-expand-features))
+                  (set! use-list (cons `(use-modules (srfi ,srfi-sym))
+                                       use-list)))))
+          extension-args)
+         (if (pair? use-list)
+             ;; i.e. (begin (use-modules x) (use-modules y) (use-modules z))
+             `(begin ,@(reverse! use-list)))))
+      (else
+       (scm-error
+        'wrong-type-arg "require-extension"
+        "Not a recognized extension type: ~S" (list extension) #f)))))
+
+\f
+
 ;;; {Load emacs interface support if emacs option is given.}
+;;;
 
 (define (named-module-use! user usee)
   (module-use! (resolve-module user) (resolve-interface usee)))
   (let ((guile-user-module (resolve-module '(guile-user))))
 
     ;; Load emacs interface support if emacs option is given.
-    (if (and (module-defined? the-root-module 'use-emacs-interface)
-            (module-ref the-root-module 'use-emacs-interface))
+    (if (and (module-defined? guile-user-module 'use-emacs-interface)
+            (module-ref guile-user-module 'use-emacs-interface))
        (load-emacs-interface))
 
     ;; Use some convenient modules (in reverse order)
     (module-use! guile-user-module (resolve-interface '(ice-9 session)))
     (module-use! guile-user-module (resolve-interface '(ice-9 debug)))
     ;; so that builtin bindings will be checked first
+    (module-use! guile-user-module (resolve-interface '(ice-9 r5rs)))
     (module-use! guile-user-module (resolve-interface '(guile)))
 
     (set-current-module guile-user-module)
                                    ;; Make a backup copy of the stack
                                    (fluid-set! before-signal-stack
                                                (fluid-ref the-last-stack))
-                                   (save-stack %deliver-signals)
+                                   (save-stack 2)
                                    (scm-error 'signal
                                               #f
                                               msg
                                  (cdr old-handler))))
                 signals old-handlers))))))
 
-(defmacro false-if-exception (expr)
-  `(catch #t (lambda () ,expr)
-         (lambda args #f)))
-
 ;;; This hook is run at the very end of an interactive session.
 ;;;
 (define exit-hook (make-hook))
 
 \f
-(append! %load-path (list "."))
 
-;; Place the user in the guile-user module.
-;;
+;;; {Deprecated stuff}
+;;;
+
+(begin-deprecated
+ (define (feature? sym)
+   (issue-deprecation-warning
+    "`feature?' is deprecated.  Use `provided?' instead.")
+   (provided? sym)))
+
+(begin-deprecated
+ (primitive-load-path "ice-9/deprecated.scm"))
+
+\f
+
+;;; Place the user in the guile-user module.
+;;;
 
 (define-module (guile-user))