Fixed (or almost so) the C lexer.

* src/guile/skribilo/coloring/Makefile.am (.l.scm): Have the generated
  module use `srfi-1'.

* src/guile/skribilo/coloring/c-lex.l: Support multi-line comments and
  properly use `*the-keys*'.

* src/guile/skribilo/coloring/c-lex.scm: Updated.

* src/guile/skribilo/coloring/c.scm: Fixed module name.
  (c-language): New.

git-archimport-id: skribilo@sv.gnu.org--2006/skribilo--devo--1.2--patch-127

5 files changed, 127 insertions, 63 deletions

diff --git a/ChangeLog b/ChangeLog
index fc6327c..81b8122 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,35 @@
 # arch-tag: automatic-ChangeLog--skribilo@sv.gnu.org--2006/skribilo--devo--1.2
 #
 
+2007-07-01 23:47:37 GMT	Ludovic Courtes <ludovic.courtes@laas.fr>	patch-127
+
+    Summary:
+      Fixed (or almost so) the C lexer.
+    Revision:
+      skribilo--devo--1.2--patch-127
+
+    * src/guile/skribilo/coloring/Makefile.am (.l.scm): Have the generated
+      module use `srfi-1'.
+    
+    * src/guile/skribilo/coloring/c-lex.l: Support multi-line comments and
+      properly use `*the-keys*'.
+    
+    * src/guile/skribilo/coloring/c-lex.scm: Updated.
+    
+    * src/guile/skribilo/coloring/c.scm: Fixed module name.
+      (c-language): New.
+
+    modified files:
+     ChangeLog src/guile/skribilo/coloring/Makefile.am
+     src/guile/skribilo/coloring/c-lex.l
+     src/guile/skribilo/coloring/c-lex.scm
+     src/guile/skribilo/coloring/c.scm
+
+    new patches:
+     lcourtes@laas.fr--2005-mobile/skribilo--devel--1.2--patch-110
+     lcourtes@laas.fr--2005-mobile/skribilo--devel--1.2--patch-111
+
+
 2007-06-29 11:45:36 GMT	Ludovic Court`es <ludovic.courtes@laas.fr>	patch-126
 
     Summary:
diff --git a/src/guile/skribilo/coloring/Makefile.am b/src/guile/skribilo/coloring/Makefile.am
index 9a3e043..5073575 100644
--- a/src/guile/skribilo/coloring/Makefile.am
+++ b/src/guile/skribilo/coloring/Makefile.am
@@ -18,6 +18,7 @@ EXTRA_DIST = lisp-lex.l xml-lex.l c-lex.l
 	echo '  :use-module (skribilo lib)' >> "$@" &&			\
 	echo '  :use-module (skribilo coloring parameters)'		\
 		>> "$@" &&						\
+	echo '  :use-module (srfi srfi-1)' >> "$@" &&			\
 	echo '  :export (lexer-init lexer' >> "$@" &&			\
 	echo '           lexer-get-func-column' >> "$@" &&		\
 	echo '           lexer-get-func-offset' >> "$@" &&		\
diff --git a/src/guile/skribilo/coloring/c-lex.l b/src/guile/skribilo/coloring/c-lex.l
index 7d7b1ce..b28a91a 100644
--- a/src/guile/skribilo/coloring/c-lex.l
+++ b/src/guile/skribilo/coloring/c-lex.l
@@ -1,28 +1,22 @@
-;;;;
-;;;; c-lex.l			-- C fontifier for Skribe
-;;;; 
-;;;; Copyright © 2004 Erick Gallesio - I3S-CNRS/ESSI <eg@essi.fr>
-;;;; 
-;;;; 
-;;;; 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 of the License, or
-;;;; (at your option) any later version.
-;;;; 
-;;;; This program is distributed in the hope that it will be useful,
-;;;; but WITHOUT ANY WARRANTY; without even the implied warranty of
-;;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-;;;; GNU General Public License for more details.
-;;;; 
-;;;; You should have received a copy of the GNU General Public License
-;;;; along with this program; if not, write to the Free Software
-;;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, 
-;;;; USA.
-;;;; 
-;;;;           Author: Erick Gallesio [eg@essi.fr]
-;;;;    Creation date:  6-Mar-2004 15:35 (eg)
-;;;; Last file update:  7-Mar-2004 00:10 (eg)
-;;;;
+;;; c-lex.l  --  C fontifier for Skribilo.
+;;;
+;;; Copyright 2004  Erick Gallesio - I3S-CNRS/ESSI <eg@essi.fr>
+;;; Copyright 2007  Ludovic Courtès <ludo@gnu.org>
+;;;
+;;; 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 of the License, or
+;;; (at your option) any later version.
+;;;
+;;; This program is distributed in the hope that it will be useful,
+;;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with this program; if not, write to the Free Software
+;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
+;;; USA.
 
 space	 [ \n\9]
 letter   [_a-zA-Z]
@@ -34,34 +28,46 @@ alphanum [_a-zA-Z0-9]
 \"[^\"]*\"		(new markup
 			     (markup '&source-string)
 			     (body yytext))
-;;Comments
-/\*.*\*/		(new markup
-			     (markup '&source-line-comment)
-			     (body   yytext))
+;; Comments
+;; FIXME: We shouldn't exclude `/' from comments but we do so to match the
+;; shortest multi-line comment.
+/\*(\n|[^/])*\*/	(let* ((not-line (char-set-complement (char-set #\newline)))
+                               (lines    (string-tokenize yytext not-line)))
+                          (reverse!
+                           (pair-fold (lambda (line* result)
+                                        (let* ((line  (car line*))
+                                               (last? (null? (cdr line*)))
+                                               (markup
+                                                (new markup
+                                                   (markup '&source-line-comment)
+                                                   (body   line))))
+                                          (if last?
+                                              (cons markup result)
+                                              (cons* (string #\newline)
+                                                     markup result))))
+                                      '()
+                                      lines)))
+
 //.*			(new markup
 			     (markup '&source-line-comment)
 			     (body   yytext))
 
 ;; Identifiers (only letters since we are interested in keywords only)
 [_a-zA-Z]+		(let* ((ident (string->symbol yytext))
-			       (tmp   (memq  ident *the-keys*)))
+			       (tmp   (memq ident (*the-keys*))))
 			  (if tmp
 			      (new markup
 				   (markup '&source-module)
 				   (body yytext))
 			      yytext))
 
-;; Regular text
-[^\"a-zA-Z]+		(begin yytext)
+;; Regular text (excluding `/' and `*')
+[^\"a-zA-Z/*]+		(begin yytext)
 
+;; `/' and `*' alone.
+/[^\*]                  (begin yytext)
+\*[^/]                  (begin yytext)
 
 
 <<EOF>>			'eof
-<<ERROR>>		(skribe-error 'lisp-fontifier "Parse error" yytext)
-
-				   
-			    
-			      
-			      
-
-			  
\ No newline at end of file
+<<ERROR>>		(skribe-error 'c-fontifier "Parse error" yytext)
diff --git a/src/guile/skribilo/coloring/c-lex.scm b/src/guile/skribilo/coloring/c-lex.scm
index 8ed6160..162c0c2 100644
--- a/src/guile/skribilo/coloring/c-lex.scm
+++ b/src/guile/skribilo/coloring/c-lex.scm
@@ -1,6 +1,7 @@
 (define-module (skribilo coloring c-lex)
   :use-module (skribilo lib)
   :use-module (skribilo coloring parameters)
+  :use-module (srfi srfi-1)
   :export (lexer-init lexer
            lexer-get-func-column
            lexer-get-func-offset
@@ -1152,7 +1153,7 @@
        ))
    (lambda (yycontinue yygetc yyungetc)
      (lambda (yytext yyline)
-         		(skribe-error 'lisp-fontifier "Parse error" yytext)
+         		(skribe-error 'c-fontifier "Parse error" yytext)
        ))
    (vector
     #t
@@ -1161,14 +1162,29 @@
           		(new markup
 			     (markup '&source-string)
 			     (body yytext))
-;;Comments
+;; Comments
+;; FIXME: We shouldn't exclude `/' from comments but we do so to match the
+;; shortest multi-line comment.
         ))
     #t
     (lambda (yycontinue yygetc yyungetc)
       (lambda (yytext yyline)
-        		(new markup
-			     (markup '&source-line-comment)
-			     (body   yytext))
+                	(let* ((not-line (char-set-complement (char-set #\newline)))
+                               (lines    (string-tokenize yytext not-line)))
+                          (reverse!
+                           (pair-fold (lambda (line* result)
+                                        (let* ((line  (car line*))
+                                               (last? (null? (cdr line*)))
+                                               (markup
+                                                (new markup
+                                                   (markup '&source-line-comment)
+                                                   (body   line))))
+                                          (if last?
+                                              (cons markup result)
+                                              (cons* (string #\newline)
+                                                     markup result))))
+                                      '()
+                                      lines)))
         ))
     #t
     (lambda (yycontinue yygetc yyungetc)
@@ -1183,36 +1199,44 @@
     (lambda (yycontinue yygetc yyungetc)
       (lambda (yytext yyline)
           		(let* ((ident (string->symbol yytext))
-			       (tmp   (memq  ident *the-keys*)))
+			       (tmp   (memq ident (*the-keys*))))
 			  (if tmp
 			      (new markup
 				   (markup '&source-module)
 				   (body yytext))
 			      yytext))
 
-;; Regular text
+;; Regular text (excluding `/' and `*')
+        ))
+    #t
+    (lambda (yycontinue yygetc yyungetc)
+      (lambda (yytext yyline)
+              		(begin yytext)
+
+;; `/' and `*' alone.
+        ))
+    #t
+    (lambda (yycontinue yygetc yyungetc)
+      (lambda (yytext yyline)
+                        (begin yytext)
         ))
     #t
     (lambda (yycontinue yygetc yyungetc)
       (lambda (yytext yyline)
-            		(begin yytext)
+                        (begin yytext)
         )))
    'decision-trees
    0
    0
-   '#((65 (35 (34 1 5) (= 47 4 1)) (96 (91 3 (95 1 2)) (97 1 (123 3 1))))
-    (65 (= 34 err 1) (97 (91 err 1) (123 err 1))) (91 (35 (34 1 err) (65 1
-    3)) (96 (95 1 2) (97 1 (123 3 1)))) (95 (65 err (91 3 err)) (97 (96 3
-    err) (123 3 err))) (47 (35 (34 1 err) (= 42 7 1)) (91 (48 6 (65 1 err))
-    (97 1 (123 err 1)))) (= 34 8 5) (35 (11 (10 6 1) (34 6 9)) (91 (65 6 9)
-    (97 6 (123 9 6)))) (42 (11 (10 7 1) (= 34 10 7)) (91 (43 11 (65 7 10))
-    (97 7 (123 10 7)))) err (= 10 err 9) (11 (10 10 err) (= 42 12 10)) (43
-    (34 (= 10 1 7) (35 10 (42 7 11))) (65 (= 47 13 7) (97 (91 10 7) (123 10
-    7)))) (42 (= 10 err 10) (47 (43 12 10) (48 14 10))) (42 (11 (10 7 1) (=
-    34 10 7)) (91 (43 11 (65 7 10)) (97 7 (123 10 7)))) (11 (10 10 err) (=
-    42 12 10)))
-   '#((#f . #f) (4 . 4) (3 . 3) (3 . 3) (4 . 4) (#f . #f) (2 . 2) (4 . 4)
-    (0 . 0) (2 . 2) (#f . #f) (4 . 4) (#f . #f) (1 . 1) (1 . 1))))
+   '#((48 (42 (= 34 6 2) (43 1 (47 2 5))) (95 (65 2 (91 4 2)) (97 (96 3 2)
+    (123 4 2)))) (= 47 err 7) (47 (35 (34 2 err) (= 42 err 2)) (91 (48 err
+    (65 2 err)) (97 2 (123 err 2)))) (48 (42 (= 34 err 2) (43 err (47 2
+    err))) (95 (65 2 (91 4 2)) (97 (96 3 2) (123 4 2)))) (95 (65 err (91 4
+    err)) (97 (96 4 err) (123 4 err))) (43 (42 8 10) (= 47 9 8)) (= 34 11
+    6) err err (= 10 err 12) (43 (42 10 13) (= 47 err 10)) err (= 10 err
+    12) (43 (42 10 13) (= 47 14 10)) err)
+   '#((#f . #f) (#f . #f) (4 . 4) (3 . 3) (3 . 3) (#f . #f) (#f . #f) (6 .
+    6) (5 . 5) (2 . 2) (#f . #f) (0 . 0) (2 . 2) (#f . #f) (1 . 1))))
 
 ;
 ; User functions
diff --git a/src/guile/skribilo/coloring/c.scm b/src/guile/skribilo/coloring/c.scm
index 28533b4..0d94307 100644
--- a/src/guile/skribilo/coloring/c.scm
+++ b/src/guile/skribilo/coloring/c.scm
@@ -19,13 +19,13 @@
 ;;; Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
 ;;; USA.
 
-(define-module (skribilo c)
+(define-module (skribilo coloring c)
   :use-module (skribilo lib)
   :use-module (skribilo utils syntax)
   :use-module (skribilo coloring c-lex)		;; SILex generated
   :use-module (skribilo coloring parameters)
   :use-module (srfi srfi-39)
-  :export (c java))
+  :export (c c-language java))
 
 (fluid-set! current-reader %skribilo-module-reader)
 
@@ -62,6 +62,10 @@
        (fontifier c-fontifier)
        (extractor #f)))
 
+(define c-language
+  ;; This alias is defined for the user's convenience.
+  c)
+
 
 ;;;
 ;;; Java.