modularize sex

.gitignore

@@ -1,5 +1,4 @@
 *.o
 *.import.scm
-*.link
 sexc
 sex-tests

Makefile

@@ -1,60 +1,30 @@
 CHICKEN_C = csc
 CSC_FLAGS += -K prefix -static
-# What and why:
-# -emit-all-import-libraries: Emit import-libraries for all defined modules.
-#    Used to generate .import.scm files so compiler would know how to use the modules.
-#    Without it, csc fails with "cannot import from undefined module" error.
-# -module-registration: Always  generate  module registration code, even when
-#    import libraries are emitted. Enables us to import from our modules at run time.
-#    Used in macros, where we inject `(import (only sex-macros cat))' before the macro body.
-#    Without it, sexc compiles scuccessfully, but is unable to process macros, failing with
-#    "during expansion of (import ...) - cannot import from undefined module: sex-macros"
-#     error.
-# -c: Stop after compilation to object files. This one is obvious.
-
-MODULE_FLAGS = -emit-all-import-libraries -module-registration -c
 
 # Order matters, since module check correctness on compilation
-MODULES = utils sex-macros reader sex-modules semen sex-fmt-c fmt-c-writer sexc
+MODULES = utils macros reader module-system semen sex-fmt-c fmt-c-writer sexc
 OBJ = $(MODULES:%=%.o)
+IMPORTS = $(MODULES:%=%.import.scm)
 
-sexc: $(OBJ) main.scm
+sexc: $(OBJ) main.o
-	$(CHICKEN_C) $(CSC_FLAGS) main.scm -o sexc-tmp -link sexc
+	$(CHICKEN_C) $(CSC_FLAGS) $^ -o $@
-        # otherwise csc hangs, probably because it tries to compile to sexc.o first
-	mv sexc-tmp sexc
 
-#------------------------------------------------------------------
+main.o: main.scm
+	$(CHICKEN_C) $(CSC_FLAGS) $< -c -o $@
 
-utils.o: utils.module.scm utils.scm
+module-system.o: module-system.scm reader.import.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) utils.module.scm -o utils.o -unit utils
+	$(CHICKEN_C) $(CSC_FLAGS) module-system.scm -e -c -J -o module-system.o -unit module-system -uses macros -uses utils
 
-sex-macros.o: sex-macros.module.scm sex-macros.scm
+%.import.scm: %.o %.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sex-macros.module.scm -o sex-macros.o -unit sex-macros
+%.o: %.scm
+	$(CHICKEN_C) $(CSC_FLAGS) $< -e -c -J -o $@ -unit $(@:%.o=%)
 
-reader.o: reader.module.scm reader.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) reader.module.scm -o reader.o -unit reader
 
-sex-modules.o: sex-modules.module.scm sex-modules.scm reader.o utils.o
+sex-tests: $(OBJ) tests/*.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sex-modules.module.scm -o sex-modules.o -unit sex-modules -link reader,utils
+	cd ./tests && $(CHICKEN_C) $(CSC_FLAGS) run.scm -c -o sex-tests.o
-
+	$(CHICKEN_C) $(CSC_FLAGS) $(OBJ) ./tests/sex-tests.o -o sex-tests
-semen.o: semen.module.scm semen.scm sex-macros.o sex-modules.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) semen.module.scm -o semen.o -unit semen -link sex-macros,sex-modules,utils
-
-sex-fmt-c.o: sex-fmt-c.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sex-fmt-c.scm -o sex-fmt-c.o -unit sex-fmt-c
-
-fmt-c-writer.o: fmt-c-writer.module.scm fmt-c-writer.scm sex-fmt-c.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) fmt-c-writer.module.scm -o fmt-c-writer.o -unit fmt-c-writer -link sex-fmt-c,utils
-
-sexc.o: sexc.module.scm sexc.scm fmt-c-writer.o sex-macros.o sex-modules.o reader.o semen.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sexc.module.scm -o sexc.o -unit sexc -link fmt-c-writer,sex-macros,sex-modules,reader,semen,utils
-
-sex-tests:
-	$(MAKE) -C tests sex-tests
-	cp ./tests/sex-tests ./
 
 clean:
-	rm -f $(OBJ) main.o
+	rm -f $(OBJ) main.o ./tests/sex-tests.o
-	rm -f *.import.scm
+	rm -f $(IMPORTS)
-	rm -f *.link
 	rm -f sexc sex-tests

fmt-c-writer.module.scm

@@ -1,3 +0,0 @@
-(module fmt-c-writer (emit-c
-                      sex-fmt-current-file)
-  "fmt-c-writer.scm")

fmt-c-writer.scm

@@ -1,6 +1,8 @@
 ;;; Sex fmt-c output writer
 
-(import
+(module fmt-c-writer (emit-c
+                      sex-fmt-current-file)
+  (import
     scheme
     (chicken base)
     (chicken string)
@@ -16,7 +18,7 @@
     tree
     utils)
 
-(define (unkebabify sym)
+  (define (unkebabify sym)
     (case sym
       ((-) sym)
       ((--) sym)
@@ -27,7 +29,7 @@
         (string-substitute "-(?!>)" "_"
                            (symbol->string sym) #t)))))
 
-(define (atom-to-fmt-c atom)
+  (define (atom-to-fmt-c atom)
     (case atom
       ((fn) '%fun)
       ((prototype) '%prototype)
@@ -47,29 +49,29 @@
            (unkebabify atom)
            atom))))
 
-(define (maybe-unwrap-type type)
+  (define (maybe-unwrap-type type)
     (if (and (list? type)
              (= 1 (length type)))
         (car type)
         type))
 
-(define (make-field-access form)
+  (define (make-field-access form)
     (assert
      (= 2 (length form)) "Wrong field access format")
     (unkebabify
      (string->symbol
       (fmt #f (cadr form) (car form)))))
 
-(define (walk-generic-toplevel form)
+  (define (walk-generic-toplevel form)
     (cond ((atom? form) (atom-to-fmt-c form))
           ((list? form) (map walk-generic-toplevel form))
           (else (error "Malformed form " form))))
 
-(define (field-access-form? form)
+  (define (field-access-form? form)
     (and (symbol? (car form))
          (char=? #\. (string-ref (symbol->string (car form)) 0))))
 
-(define (walk-expr form)
+  (define (walk-expr form)
     (match form
       ((? vector?)
        (list->vector
@@ -90,7 +92,7 @@
       (('c-bit-or= . rest) (apply c-bit-or= (map walk-expr rest)))
       (else (map walk-expr form))))
 
-(define (walk-var form)
+  (define (walk-var form)
     ;; (var a int) -> (%var int a)
     ;; (var a (const int) 32) -> (%var (const int) a 32)
     ;; (var b [const char 512]) -> (%var (%array (const char) 512) b)
@@ -105,7 +107,7 @@
            (walk-expr (drop form 3)))      ; optional init expression
       ))
 
-(define (walk-type form)
+  (define (walk-type form)
     ;; int -> int
     ;; (const int) -> const int
     ;; [const char 512] ->(¤ (const char) 512) -> (%array (const char) 512)
@@ -138,7 +140,7 @@
       (else
        (type-convert-to-c form))))
 
-(define (type-convert-to-c type)
+  (define (type-convert-to-c type)
     ;; Our pointers to C pointers
     ;; int -> int
     ;; * const char -> const char *
@@ -150,7 +152,7 @@
                     (list-join (reverse (list-split type '*))
                                '(*)))))))
 
-(define (walk-fn-def form)
+  (define (walk-fn-def form)
     (match form
       (('fn name args ret-type . maybe-body)
        `(%fun
@@ -161,12 +163,12 @@
          ,(walk-expr maybe-body)))))
 
 ;;; TODO: isn't there a better way?
-(define (is-probably-type form)
+  (define (is-probably-type form)
     (case (car form)
       ((¤ * const volatile struct union) #t)
       (else #f)))
 
-(define (walk-arglist form)
+  (define (walk-arglist form)
     ;; E.g.:
     ;; ((float) (int) (const char) (* const char) (¤ (* const struct res) 32))
     ;; ((f1 float) (f2 float) (f3 float) (res (¤ float 4)))
@@ -186,20 +188,20 @@
                                   (list (walk-type var))))))
          form))
 
-(define (walk-function form)
+  (define (walk-function form)
     ;; (fn ret-type name arglist body) -> normal function
     ;; (fn ret-type name arglist) -> prototype
     (if (>= (length form) 5)
         (walk-fn-def form)
         (cons '%prototype (cdr (walk-fn-def form)))))
 
-(define (process-struct-fields fields)
+  (define (process-struct-fields fields)
     (map (fn
           (let ((type (walk-type (last x))))
             (cons type (map atom-to-fmt-c (drop-right x 1)))))
          fields))
 
-(define (walk-struct form)
+  (define (walk-struct form)
     (match form
       ((type (fields ...) . attrs)        ; anonymous struct
        `(,type ,(process-struct-fields fields)
@@ -212,14 +214,14 @@
                . ,(tree-map atom-to-fmt-c attrs)))
       (else (error "Malformed aggregate definition " form))))
 
-(define (walk-enum form)
+  (define (walk-enum form)
     (match form
       (('enum (values ...))
        `(enum ,(map atom-to-fmt-c values)))
       (('enum name (values ...))
        `(enum ,(atom-to-fmt-c name) ,(map atom-to-fmt-c values)))))
 
-(define (walk-extern form)
+  (define (walk-extern form)
     (match form
       (('fn . _)
        ;; extern function?.. What
@@ -228,7 +230,7 @@
        (list 'extern (walk-var form)))
       (else (error "Extern what?"))))
 
-(define (walk-public form)
+  (define (walk-public form)
     (match form
       (('fn . _)
        (walk-function form))
@@ -249,7 +251,7 @@
       (else
        (error "Pub what?" (cadr form)))))
 
-(define (process-toplevel-form form)
+  (define (process-toplevel-form form)
     (match form
       (('fn . _) (list 'static (walk-function form)))
       (('var . _) (list 'static (walk-var form)))
@@ -261,23 +263,23 @@
       (('define name . rest) `(%define ,(atom-to-fmt-c name) ,@rest))
       (else (walk-expr form))))
 
-(define (get-line-num form)
+  (define (get-line-num form)
     (let ((num (get-line-number form)))
       (if (string? num)
           (last (string-split num ":"))
           #f)))
 
-(define sex-fmt-current-file (make-parameter "/dev/null"))
+  (define sex-fmt-current-file (make-parameter "/dev/null"))
-(define sex-fmt-line-num (make-parameter 0))
+  (define sex-fmt-line-num (make-parameter 0))
 
-(define (line-directive-string)
+  (define (line-directive-string)
     (fmt #f #\# "line " (sex-fmt-line-num) " " #\" (sex-fmt-current-file) #\"))
 
-(define (emit-c sex-forms)
+  (define (emit-c sex-forms)
     (for-each (lambda (form)
                 (let ((start-line (get-line-num form)))
                   (when start-line
                     (sex-fmt-line-num start-line)
                     (fmt #t (line-directive-string) nl)))
                 (fmt #t (c-expr (process-toplevel-form form)) nl))
-            sex-forms))
+              sex-forms)))

macros.scm

@@ -0,0 +1,45 @@
+(module macros
+    (register-macro
+     cat
+     get-macro
+     macro?
+     apply-macro
+     defmacro)
+  (import
+    scheme
+    (only fmt fmt)
+    (chicken base)
+    (chicken plist)
+    (chicken string))
+
+  (define (cat-syms s-1 s-2)
+    (fmt #f s-1 s-2))
+
+  (define (cat sym-1 sym-2)
+    (string->symbol (cat-syms sym-1 sym-2)))
+
+  (define (register-macro name arglist body)
+    (put! name 'sex-macro
+          `(lambda ,arglist
+             (import scheme
+                     (only macros cat))
+             ,@body)))
+
+  (define (get-macro name)
+    (eval (get name 'sex-macro)))
+
+  (define (macro? form)
+    (and (list? form)
+         (symbol? (car form))
+         (get (car form) 'sex-macro)))
+
+  (define (apply-macro form)
+    (assert (macro? form)
+            (fmt #f (car form) " is not a macro"))
+    (apply (get-macro (car form))
+           (cdr form)))
+
+  (define (defmacro form)
+    (let ((arglist (car form))
+          (body (cdr form)))
+      (register-macro (car arglist) (cdr arglist) body))))

module-system.scm

@@ -0,0 +1,92 @@
+(module module-system
+    (get-modules-public-forms
+     load-persistent-module-paths
+     read-public-interface)
+
+  (import
+    scheme
+    brev-separate
+    (chicken base)
+    (chicken file)
+    (chicken load)
+    (chicken pathname)
+    (chicken process-context)
+    (chicken string)
+    fmt
+    reader
+    srfi-1
+    utils)
+
+  (define +persistent-module-paths+ (list))
+
+  (define (get-modules-public-forms module-list)
+    ;; Module list is a list of symbols
+    ;; How Sex handles modules:
+    ;; For each module in a list, construct path, find module by path in
+    ;; module path directories, extract public definitions from the
+    ;; module, paste them in current one in emulation of C include
+    ;; directives.
+    (fold-right append (list)
+                (map (fn (import-module (symbol->string x)))
+                     module-list)))
+
+  (define (import-module name)
+    (let ((module-path (locate-module name)))
+      (assert module-path (fmt #f "Failed to find module " name " in "
+                               (get-module-paths)))
+      (read-public-interface module-path)))
+
+  (define (get-module-paths)
+    (cons (current-directory)
+          +persistent-module-paths+))
+
+  (define (locate-module name)
+    ;; Module locations: relative to file being compiled, or in what was
+    ;; in SEX_MODULE_PATH env var at the start of the process (see
+    ;; load-persistent-module-paths function)
+
+    (let ((search-paths (get-module-paths)))
+      (let loop ((paths search-paths))
+        (if (null? paths)
+            #f
+            (or (module-exists? name (car paths))
+                (loop (cdr paths)))))))
+
+  (define (module-exists? name module-dir)
+    ;; returns absolute path to module, if it exists
+    (and (directory-exists? module-dir)
+         (let ((module-path (make-absolute-pathname module-dir name "sex")))
+           (and (file-exists? module-path)
+                (file-readable? module-path)
+                module-path))))
+
+  (define (read-public-interface module-path)
+    ;; pub fns are reduced to prototypes, other pub forms are just pasted
+    (let ((raw-forms (read-from-file module-path)))
+      (fold
+       process-public-interface-form
+       (list)
+       raw-forms)))
+
+;;; TODO: use semen facilities to analyze modules
+  (define (process-public-interface-form form acc)
+    (case (car form)
+      ((pub)
+       (case (cadr form)
+         ((fn)                 ; replace with prototype
+          ;; fn type name (arg-list) (body)
+          ;; 1    2   3       4 - we need first 4
+          (cons (take (cdr form) 4) acc))
+         ((define defmacro import include struct typedef union var)
+          (cons (cdr form) acc))
+         (else (error "Pub what? " (cadr form)))))
+      (else acc)))
+
+  (define (load-persistent-module-paths)
+    (let ((sex-module-path-env-var
+           (get-env-var "SEX_MODULE_PATH")))
+      (when sex-module-path-env-var
+        (set! +persistent-module-paths+
+          (map (lambda (p)
+                 (make-absolute-pathname p #f #f))
+               (string-split sex-module-path-env-var ":")))))))

reader.module.scm

@@ -1,3 +0,0 @@
-(module reader (read-from-file
-                read-raw-forms)
-  "reader.scm")

reader.scm

@@ -1,4 +1,6 @@
-(import
+(module reader (read-from-file
+                read-raw-forms)
+  (import
     scheme
     (chicken base)
     (chicken io)
@@ -10,19 +12,19 @@
     brev-separate
     fmt)
 
-(import-syntax utils)
+  (import-syntax utils)
 
-(define (read-forms acc)
+  (define (read-forms acc)
     (let ((r (read-with-source-info (current-input-port))))
       (if (eof-object? r) (reverse acc)
           (read-forms (cons r acc)))))
 
-(define (read-from-file file)
+  (define (read-from-file file)
     (with-directory file
                     (with-input-from-file (pathname-strip-directory file)
                       (fn (read-forms (list))))))
 
-(define (read-bracket port)
+  (define (read-bracket port)
     (let loop ((c (read-char port))
                (str (string)))
       (cond ((char=? c #\])
@@ -35,9 +37,9 @@
              (loop (read-char port)
                    (conc str c))))))
 
-(define open-bracket-counter (make-parameter 0))
+  (define open-bracket-counter (make-parameter 0))
 
-(define (read-raw-forms input-source)
+  (define (read-raw-forms input-source)
     (let ((bracket-end (gensym)))
       (set-read-syntax!
        #\]
@@ -59,4 +61,4 @@
                      (cons r acc)))))))
     (if (eq? input-source 'stdin)
         (read-forms (list))
-      (read-from-file input-source)))
+        (read-from-file input-source))))

semen.module.scm

@@ -1,2 +0,0 @@
-(module semen ()
-  "semen.scm")

semen.scm

@@ -1,21 +1,22 @@
 ;;; Sex semantic engine
 
-(import
+(module semen ()
+  (import
     scheme
     (chicken base)
     (chicken keyword)
     (chicken string)
     (chicken module)
     fmt
-  sex-macros
+    macros
-  sex-modules
     matchable                         ; pattern matching
+    module-system
     srfi-1                            ; list routines
     srfi-69                           ; hash tables
     utils
     )
 
-(export/rename (process semen-process))
+  (export/rename (process semen-process))
 
 ;;; for lambda extraction, docstring processing, macro expansion,
 ;;; injection of module headers, i.e. all things that rearrange code
@@ -25,10 +26,10 @@
 ;;; append their return to the resulting list. Each handler can return
 ;;; multiple forms, e.g. lambdas collected from a function may result
 ;;; in auxiliary structures and functions.
-(define (process raw-sex-forms)
+  (define (process raw-sex-forms)
     (process-rec raw-sex-forms (list)))
 
-(define (process-rec forms acc)
+  (define (process-rec forms acc)
     (cond
      ((null? forms) (reverse acc))
      ((macro? (car forms))
@@ -39,7 +40,7 @@
       (process-rec (cdr forms)
                    (match-sex-form (car forms) acc)))))
 
-(define (macroexpand macro-form rest-forms)
+  (define (macroexpand macro-form rest-forms)
     ;; We want to replace macro with its expansion. The problem is,
     ;; top-level macro can return either a single form, or a list of
     ;; forms, when it for example generates some aux
@@ -52,7 +53,7 @@
           (append res rest-forms)
           (cons res rest-forms))))
 
-(define (match-sex-form sex-form acc)
+  (define (match-sex-form sex-form acc)
     (match sex-form
       ((or ('fn . _)
            ('pub 'fn . _)
@@ -81,7 +82,7 @@
 
       (else (assert #f (fmt #f "Unknown top level form " sex-form)))))
 
-(define (process-imports module-public-forms acc)
+  (define (process-imports module-public-forms acc)
     ;; Recursively process imports: register public macros, cons all
     ;; other public things to our acc
     (if (null? module-public-forms) acc
@@ -94,7 +95,7 @@
                             (cons (car module-public-forms)
                                   acc))))))
 
-(define (macro-expand form)
+  (define (macro-expand form)
     "Walk the form recursively and expand all macros, until none is left."
     (walk-form
      form
@@ -112,12 +113,12 @@
 ;;; For inspiration, see SBCL's walk.lisp and their template
 ;;; system.
 
-(define walk-embed-result (gensym)
+  (define walk-embed-result (gensym)
     ;; For cases when result is a list which must be embedded in the
     ;; form, e.g. when it returned from a macro
     )
 
-(define (walk-form form walk-fn env)
+  (define (walk-form form walk-fn env)
     (if (atom? form) form
         (let ((new-form (walk-fn form env)))
           (cond ((not (eq? form new-form))
@@ -133,12 +134,12 @@
 
 ;;; Typdef
 
-(define (process-typedef new-type target acc)
+  (define (process-typedef new-type target acc)
     (cons `(typedef ,target ,new-type) acc))
 
 ;;; Fn processing
 
-(define (process-fn sex-fn acc)
+  (define (process-fn sex-fn acc)
     (let* ((expanded (macro-expand sex-fn))
            (env (make-hash-table))
            (processed
@@ -154,7 +155,7 @@
       (cons processed
             (append (hash-table-ref env :lambda-aux-code) acc))))
 
-(define (fn-walker form env)
+  (define (fn-walker form env)
     (if (eq? 'lambda (car form))
         (let ((lambda-name (make-lambda-name (hash-table-ref env :fn-name)
                                              (hash-table-ref env :lambda-counter))))
@@ -166,11 +167,11 @@
           lambda-name)
         form))
 
-(define (make-lambda-name enclosing-fn-name counter)
+  (define (make-lambda-name enclosing-fn-name counter)
     (string->symbol
      (fmt #f "__lambda_" counter "_" enclosing-fn-name)))
 
-(define (make-aux-lambda-struct name form)
+  (define (make-aux-lambda-struct name form)
     (match form
       (('lambda ret-type arglist captures . body)
        ;; Captures are ignored for now, but
@@ -180,18 +181,18 @@
 
 ;;; Structs
 
-(define (process-struct sex-struct acc)
+  (define (process-struct sex-struct acc)
     (cons sex-struct acc))
 
-(define (process-global-var sex-var acc)
+  (define (process-global-var sex-var acc)
     (cons sex-var acc))
 
 ;;; Utils
-(define (non-empty-list? form)
+  (define (non-empty-list? form)
     (and (list? form)
          (not (null? form))))
 
-(define (sex-fn? form)
+  (define (sex-fn? form)
     "The `form` must be toplevel.
 Returns #f if the form is not a function, returns the form otherwise"
     (match form
@@ -199,31 +200,31 @@ Returns #f if the form is not a function, returns the form otherwise"
       ((pub fn . _) form)
       (else #f)))
 
-(define (sex-fn-public? fn-form)
+  (define (sex-fn-public? fn-form)
     (eq? (car fn-form) 'pub))
 
-(define (sex-fn-return-type fn-form)
+  (define (sex-fn-return-type fn-form)
     (assert (sex-fn? fn-form)
             (fmt #f "Form " fn-form " is not a function"))
     (if (sex-fn-public? fn-form)
         (third fn-form)
         (second fn-form)))
 
-(define (sex-fn-name fn-form)
+  (define (sex-fn-name fn-form)
     (assert (sex-fn? fn-form)
             (fmt #f "Form " fn-form " is not a function"))
     (if (sex-fn-public? fn-form)
         (fourth fn-form)
         (third fn-form)))
 
-(define (sex-fn-arglist fn-form)
+  (define (sex-fn-arglist fn-form)
     (assert (sex-fn? fn-form)
             (fmt #f "Form " fn-form " is not a function"))
     (if (sex-fn-public? fn-form)
         (fifth fn-form)
         (fourth fn-form)))
 
-(define (sex-fn-prototype fn-form)
+  (define (sex-fn-prototype fn-form)
     "Returns all except body"
     (assert (sex-fn? fn-form)
             (fmt #f "Form " fn-form " is not a function"))
@@ -231,9 +232,10 @@ Returns #f if the form is not a function, returns the form otherwise"
         (take fn-form 5)
         (take fn-form 4)))
 
-(define (sex-fn-body fn-form)
+  (define (sex-fn-body fn-form)
     (assert (sex-fn? fn-form)
             (fmt #f "Form " fn-form " is not a function"))
     (if (sex-fn-public? fn-form)
         (drop fn-form 5)
         (drop fn-form 4)))
+  )

sex-fmt-c.scm

@@ -112,7 +112,7 @@
     (lambda (st)
       ((fmt-let 'op op
                 (if (and (c-op<= (fmt-op st) op)
-                         (not (vector? x)))
+                         (not (vector? st)))
                     (c-paren x)
                     x))
        st)))

sex-macros.module.scm

@@ -1,8 +0,0 @@
-(module sex-macros
-    (register-macro
-     cat
-     get-macro
-     macro?
-     apply-macro
-     defmacro)
-  "sex-macros.scm")

sex-macros.scm

@@ -1,38 +0,0 @@
-(import
-  scheme
-  (only fmt fmt)
-  (chicken base)
-  (chicken plist)
-  (chicken string))
-
-(define (cat-syms s-1 s-2)
-  (fmt #f s-1 s-2))
-
-(define (cat sym-1 sym-2)
-  (string->symbol (cat-syms sym-1 sym-2)))
-
-(define (register-macro name arglist body)
-  (put! name 'sex-macro
-        `(lambda ,arglist
-           (import scheme
-                   (only sex-macros cat))
-           ,@body)))
-
-(define (get-macro name)
-  (eval (get name 'sex-macro)))
-
-(define (macro? form)
-  (and (list? form)
-       (symbol? (car form))
-       (get (car form) 'sex-macro)))
-
-(define (apply-macro form)
-  (assert (macro? form)
-          (fmt #f (car form) " is not a macro"))
-  (apply (get-macro (car form))
-         (cdr form)))
-
-(define (defmacro form)
-  (let ((arglist (car form))
-        (body (cdr form)))
-    (register-macro (car arglist) (cdr arglist) body)))

sex-mode.el

@@ -51,9 +51,7 @@ All commands in `lisp-mode-shared-map' are inherited by this map.")
      ;; Keywords
      (list (concat "("
                    (regexp-opt '(
-                                 "begin"
                                  "case"
-                                 "default"
                                  "do"
                                  "if"
                                  "for"
@@ -85,8 +83,6 @@ All commands in `lisp-mode-shared-map' are inherited by this map.")
 (put 'union 'lisp-indent-function 'defun)
 (put 'var 'lisp-indent-function 0)
 (put 'import 'lisp-indent-function 1)
-(put 'switch 'lisp-indent-function 1)
-(put 'case 'lisp-indent-function 1)
 
 ;;;###autoload
 (define-derived-mode sex-mode lisp-data-mode "Sex"

sex-modules.module.scm

@@ -1,5 +0,0 @@
-(module sex-modules
-    (get-modules-public-forms
-     load-persistent-module-paths
-     read-public-interface)
-  "sex-modules.scm")

sex-modules.scm

@@ -1,87 +0,0 @@
-(import
-  scheme
-  brev-separate
-  (chicken base)
-  (chicken file)
-  (chicken load)
-  (chicken pathname)
-  (chicken process-context)
-  (chicken string)
-  fmt
-  reader
-  srfi-1
-  utils)
-
-(define +persistent-module-paths+ (list))
-
-(define (get-modules-public-forms module-list)
-  ;; Module list is a list of symbols
-  ;; How Sex handles modules:
-  ;; For each module in a list, construct path, find module by path in
-  ;; module path directories, extract public definitions from the
-  ;; module, paste them in current one in emulation of C include
-  ;; directives.
-  (fold-right append (list)
-              (map (fn (import-module (symbol->string x)))
-                   module-list)))
-
-(define (import-module name)
-  (let ((module-path (locate-module name)))
-    (assert module-path (fmt #f "Failed to find module " name " in "
-                             (get-module-paths)))
-    (read-public-interface module-path)))
-
-(define (get-module-paths)
-  (cons (current-directory)
-        +persistent-module-paths+))
-
-(define (locate-module name)
-  ;; Module locations: relative to file being compiled, or in what was
-  ;; in SEX_MODULE_PATH env var at the start of the process (see
-  ;; load-persistent-module-paths function)
-
-  (let ((search-paths (get-module-paths)))
-    (let loop ((paths search-paths))
-      (if (null? paths)
-          #f
-          (or (module-exists? name (car paths))
-              (loop (cdr paths)))))))
-
-(define (module-exists? name module-dir)
-  ;; returns absolute path to module, if it exists
-  (and (directory-exists? module-dir)
-       (let ((module-path (make-absolute-pathname module-dir name "sex")))
-         (and (file-exists? module-path)
-              (file-readable? module-path)
-              module-path))))
-
-(define (read-public-interface module-path)
-  ;; pub fns are reduced to prototypes, other pub forms are just pasted
-  (let ((raw-forms (read-from-file module-path)))
-    (fold
-     process-public-interface-form
-     (list)
-     raw-forms)))
-
-;;; TODO: use semen facilities to analyze modules
-(define (process-public-interface-form form acc)
-  (case (car form)
-    ((pub)
-     (case (cadr form)
-       ((fn)                 ; replace with prototype
-        ;; fn type name (arg-list) (body)
-        ;; 1    2   3       4 - we need first 4
-        (cons (take (cdr form) 4) acc))
-       ((define defmacro import include struct typedef union var)
-        (cons (cdr form) acc))
-       (else (error "Pub what? " (cadr form)))))
-    (else acc)))
-
-(define (load-persistent-module-paths)
-  (let ((sex-module-path-env-var
-         (get-env-var "SEX_MODULE_PATH")))
-    (when sex-module-path-env-var
-      (set! +persistent-module-paths+
-        (map (lambda (p)
-               (make-absolute-pathname p #f #f))
-             (string-split sex-module-path-env-var ":"))))))

sexc.module.scm

@@ -1 +0,0 @@
-(module sexc (main) "sexc.scm")

sexc.scm

@@ -1,4 +1,5 @@
-(import scheme
+(module sexc (main)
+  (import scheme
           brev-separate
           (chicken base)
           (chicken file)
@@ -10,8 +11,8 @@
           fmt
           fmt-c-writer
           getopt-long
-        sex-macros
+          macros
-        sex-modules
+          module-system
           reader
           semen
           srfi-1                          ; list routines
@@ -21,7 +22,7 @@
 
 ;;; Main function facilities
 
-(define opts-grammar
+  (define opts-grammar
     (let ((padding 26))
       `((c-compiler ,(fmt #f "Select C compiler. Defaults to value of SEX_CC" nl
                           (pad padding) "environment variable, or if it is empty, to cc")
@@ -52,46 +53,46 @@
                 (value #t)
                 (single-char #\o)))))
 
-(define (print-help)
+  (define (print-help)
     (fmt #t "Usage: sexc [options] filename [-- options-for-c-compiler]\n")
     (fmt #t "Options:\n")
     (fmt #t (usage opts-grammar))
     (fmt #t ""))
 
-(define (help-arg? args)
+  (define (help-arg? args)
     (assoc 'help args))
 
-(define (get-arg args arg-name default)
+  (define (get-arg args arg-name default)
     (let ((arg (assoc arg-name args)))
       (if arg (cdr arg)
           default)))
 
-(define (get-rest-args args)
+  (define (get-rest-args args)
     (cdr (assoc '@ args)))
 
-(define (get-c-compiler-args args)
+  (define (get-c-compiler-args args)
     (filter (fn (string-prefix? "-" x)) (get-rest-args args)))
 
-(define (get-input-file args)
+  (define (get-input-file args)
     (let ((rest-args (get-rest-args args)))
       (if (null? rest-args)
           'stdin
           (car rest-args))))
 
-(define (write-to-file-or-stdout output what)
+  (define (write-to-file-or-stdout output what)
     (if (eq? output 'default)
         (what)
         (with-output-to-file output
           (fn (what)))))
 
-(define (emit-c-or-sex sex-forms output args)
+  (define (emit-c-or-sex sex-forms output args)
     (write-to-file-or-stdout output
                              (lambda ()
                                (if (get-arg args 'macro-expand #f)
                                    (map pp sex-forms)
                                    (emit-c sex-forms)))))
 
-(define (compile-to-file sex-forms output args)
+  (define (compile-to-file sex-forms output args)
     (let ((compiler (or (get-arg args 'c-compiler #f)
                         (get-env-var "SEX_CC")
                         "cc"))
@@ -112,13 +113,13 @@
           (close-output-port in-port)
           (process-wait pid)))))
 
-(define (semantic-process-forms raw-forms input-source)
+  (define (semantic-process-forms raw-forms input-source)
     (if (eq? input-source 'stdin)
         (semen-process raw-forms)
         (with-directory input-source
                         (semen-process raw-forms))))
 
-(define prelude
+  (define prelude
     '((include inttypes.h)
 
       (typedef u8 uint8-t)
@@ -130,7 +131,7 @@
       (typedef u64 uint64-t)
       (typedef i64 int64-t)))
 
-(define (main)
+  (define (main)
     (let* ((raw-args (command-line-arguments))
            (args (getopt-long raw-args
                               opts-grammar))
@@ -163,4 +164,4 @@
                ;; Emit processed and macro-expanded sex code, or emit C code
                (emit-c-or-sex sex-forms output args)
                ;; Compile file!
-             (compile-to-file sex-forms output args)))))))
+               (compile-to-file sex-forms output args))))))))

tests/Makefile

@@ -1,45 +1 @@
-CHICKEN_C = csc
 
-CSC_FLAGS += -K prefix -static
-MODULE_FLAGS = -emit-all-import-libraries -module-registration -c
-
-MODULES = utils sex-macros reader sex-modules semen sex-fmt-c fmt-c-writer sexc
-SEX_OBJ = $(MODULES:%=%.o)
-
-TESTS = basic semen reader fmt-c-writer utils
-TEST_SRCS = $(TESTS:%=%.scm)
-
-sex-tests: run.scm $(TEST_SRCS) $(SEX_OBJ)
-	$(CHICKEN_C) $(CSC_FLAGS) run.scm -o sex-tests -link sexc
-
-#------------------------------------------------------------------
-
-utils.o: utils.module.scm ../utils.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) utils.module.scm -o utils.o -unit utils
-
-sex-macros.o: sex-macros.module.scm ../sex-macros.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sex-macros.module.scm -o sex-macros.o -unit sex-macros
-
-reader.o: reader.module.scm ../reader.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) reader.module.scm -o reader.o -unit reader
-
-sex-modules.o: sex-modules.module.scm ../sex-modules.scm reader.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sex-modules.module.scm -o sex-modules.o -unit sex-modules -link reader,utils
-
-semen.o: semen.module.scm ../semen.scm sex-macros.o sex-modules.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) semen.module.scm -o semen.o -unit semen -link sex-macros,sex-modules,utils
-
-sex-fmt-c.o: ../sex-fmt-c.scm
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) ../sex-fmt-c.scm -o sex-fmt-c.o -unit sex-fmt-c
-
-fmt-c-writer.o: fmt-c-writer.module.scm ../fmt-c-writer.scm sex-fmt-c.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) fmt-c-writer.module.scm -o fmt-c-writer.o -unit fmt-c-writer -link sex-fmt-c,utils
-
-sexc.o: sexc.module.scm ../sexc.scm fmt-c-writer.o sex-macros.o sex-modules.o reader.o semen.o utils.o
-	$(CHICKEN_C) $(CSC_FLAGS) $(MODULE_FLAGS) sexc.module.scm -o sexc.o -unit sexc -link fmt-c-writer,sex-macros,sex-modules,reader,semen,utils
-
-clean:
-	rm -f $(OBJ)
-	rm -f *.import.scm
-	rm -f *.link
-	rm -f sex-tests

tests/basic.scm

@@ -1,5 +1,3 @@
-(import fmt-c-writer)
-
 (test-group "basic"
 
   ;; unkebabify

tests/fmt-c-writer.module.scm

@@ -1,3 +0,0 @@
-(module fmt-c-writer
-    *
-  "../fmt-c-writer.scm")

tests/reader.module.scm

@@ -1,3 +0,0 @@
-(module reader
-    *
-  "../reader.scm")

tests/reader.scm

@@ -1,5 +1,4 @@
-(import (chicken port)
+(import (chicken port))
-        reader)
 
 (define-syntax reader-test
   (syntax-rules ()

tests/run.scm

@@ -1,4 +1,10 @@
+(declare (uses fmt-c-writer
+               semen))
+
 (import
+  (chicken process)
+  (chicken process-context)
+  srfi-1
   test)
 
 (include "basic.scm")

tests/semen.module.scm

@@ -1,2 +0,0 @@
-(module semen *
-  "../semen.scm")

tests/semen.scm

@@ -1,5 +1,4 @@
-(import srfi-69
+(import srfi-69)
-        semen)
 
 (define print-str-fn
   '(fn void print-str ((string s))
@@ -39,11 +38,11 @@
   (define (form-identity form env)
     form)
 
-  (test 'a (walk-form 'a form-identity (make-hash-table)))
+  (test 'a (semen-walk-form 'a form-identity (make-hash-table)))
-  (test '(a b c) (walk-form '(a b c) form-identity (make-hash-table)))
+  (test '(a b c) (semen-walk-form '(a b c) form-identity (make-hash-table)))
 
-  (test 'a (macro-expand 'a))
+  (test 'a (semen-macro-expand 'a))
-  (test '(a b c) (macro-expand '(a b c)))
+  (test '(a b c) (semen-macro-expand '(a b c)))
 
   (let ((sex-code-macro
          '((defmacro (x10 a)

tests/sex-macros.module.scm

@@ -1,3 +0,0 @@
-(module sex-macros
-    *
-  "../sex-macros.scm")

tests/sex-modules.module.scm

@@ -1,3 +0,0 @@
-(module sex-modules
-    *
-  "../sex-modules.scm")

tests/sexc.module.scm

@@ -1 +0,0 @@
-(module sexc (main) "../sexc.scm")

tests/utils.module.scm

@@ -1,3 +0,0 @@
-(module utils
-    *
-  "../utils.scm")

tests/utils.scm

@@ -1,5 +1,3 @@
-(import utils)
-
 (test-group "utils"
 
   (test

utils.macros.scm

@@ -0,0 +1 @@
+(import (chicken process-context))

utils.module.scm

@@ -1,10 +0,0 @@
-(module utils
-    (get-env-var
-     set-working-directory
-     to-absolute-pathname
-     list-split
-     list-join
-     recons
-     with-directory
-     )
-  "utils.scm")

utils.scm

@@ -1,18 +1,28 @@
-(import
+(module utils
+    (get-env-var
+     set-working-directory
+     to-absolute-pathname
+     list-split
+     list-join
+     recons
+     with-directory
+     )
+
+  (import
     scheme
     (chicken base)
     (chicken pathname)
     (chicken process-context)
     srfi-1)
 
-(define-syntax prog1
+  (define-syntax prog1
     (syntax-rules ()
       ((prog1 form . forms)
        (let ((res form))
          (begin . forms)
          res))))
 
-(define-syntax with-directory
+  (define-syntax with-directory
     (syntax-rules ()
       ((with-directory path form . forms)
        (let ((current-dir (current-directory)))
@@ -21,10 +31,10 @@
           (begin form . forms)
           (change-directory current-dir))))))
 
-(define (get-env-var name)
+  (define (get-env-var name)
     (get-environment-variable name))
 
-(define (set-working-directory file)
+  (define (set-working-directory file)
     (change-directory
      (normalize-pathname
       (if (absolute-pathname? file)
@@ -33,14 +43,14 @@
            (current-directory)
            (pathname-directory file))))))
 
-(define (to-absolute-pathname pathname)
+  (define (to-absolute-pathname pathname)
     (if (absolute-pathname? pathname)
         pathname
         (make-absolute-pathname
          (current-directory)
          pathname)))
 
-(define (list-split src-list split-elt)
+  (define (list-split src-list split-elt)
     ;; split '(1 2 / 3 4 / 5 6) by '/ -> '((1 2) (3 4) (5 6))
     (fold (lambda (elt acc)
             (if (eq? elt split-elt)
@@ -50,7 +60,7 @@
           (list (list))
           src-list))
 
-(define (list-join lists join-by)
+  (define (list-join lists join-by)
     (drop-right
      (fold (lambda (elt acc)
              (append acc (list elt) (list join-by)))
@@ -59,8 +69,9 @@
      1))
 
 ;;; Reconstruct form
-(define (recons old-cons new-car new-cdr)
+  (define (recons old-cons new-car new-cdr)
     (if (and (eq? new-car (car old-cons))
              (eq? new-cdr (cdr old-cons)))
         old-cons
         (cons new-car new-cdr)))
+  )