X-Git-Url: http://git.kpe.io/?a=blobdiff_plain;f=parse-common.lisp;h=28a57db74ed78405a9b352558c4426f30d655ebe;hb=5be6bdf06e0c99612a4c1a1d6f110e424400d96e;hp=198041445fcf9baf870f8ac45a28af19436dc64c;hpb=dff3199405205cf99782dd3abf9d9dde187f5494;p=umlisp.git

diff --git a/parse-common.lisp b/parse-common.lisp
index 1980414..28a57db 100644
--- a/parse-common.lisp
+++ b/parse-common.lisp
@@ -7,10 +7,10 @@
 ;;;; Author:        Kevin M. Rosenberg
 ;;;; Date Started:  Apr 2000
 ;;;;
-;;;; $Id: parse-common.lisp,v 1.10 2003/05/07 22:53:36 kevin Exp $
+;;;; $Id$
 ;;;;
 ;;;; This file, part of UMLisp, is
-;;;;    Copyright (c) 2000-2002 by Kevin M. Rosenberg, M.D.
+;;;;    Copyright (c) 2000-2003 by Kevin M. Rosenberg, M.D.
 ;;;;
 ;;;; UMLisp users are granted the rights to distribute and use this software
 ;;;; as governed by the terms of the GNU General Public License.
@@ -21,7 +21,7 @@
 (eval-when (:compile-toplevel)
   (declaim (optimize (speed 3) (safety 1) (compilation-speed 0) (debug 3))))
 
-(defun ensure-init-umls (&optional (alwaysclear nil))
+(defun ensure-ucols+ufiles (&optional (alwaysclear nil))
 "Initialize all UMLS file and column structures if not already initialized"
   (when (or alwaysclear (null *umls-files*))
     (gen-ucols)
@@ -72,7 +72,7 @@ Currently, these are the LEX and NET files."
     (destructuring-bind (filename fields-max fields-av) length-list
       (let ((file (find-ufile filename)))
 	(unless file
-	  (error "Can't find ~A filename in ufiles"))
+	  (error "Can't find ~A filename in ufiles" filename))
 	(unless (= (length fields-max) (length (fields file)))
 	  (error
 	   "Number of file fields ~A not equal to field count in ufile ~S" 
@@ -162,6 +162,8 @@ Currently, these are the LEX and NET files."
 (defun ensure-compiled-fun (fun)
   "Ensure that a function is compiled"
   (etypecase fun
+    (null
+     nil)
     (function
      (if (compiled-function-p fun)
 	 fun
@@ -174,7 +176,9 @@ Currently, these are the LEX and NET files."
 		  (quote-str "'") (custom-value-fun))
   (let ((ucol (make-instance
 	       'ucol
-	       :col col :des des :ref ref :min min :av av :max max :fil fil
+	       :col col :des des :ref ref :min min :av av 
+	       :max (if (eql max 0) 1 max) ;; ensure at least one char wide
+	       :fil fil
 	       :dty dty :sqltype sqltype :quote-str quote-str
 	       :parse-fun (ensure-compiled-fun parse-fun)
  	       :custom-value-fun (ensure-compiled-fun custom-value-fun))))
@@ -182,7 +186,7 @@ Currently, these are the LEX and NET files."
     ucol))
 
 (defun make-empty-ucol (colname filename)
-  (warn "call in make-empty-ucol")
+  ;;(format "call in make-empty-ucol: ~A/~A" colname filename)
   (make-ucol (copy-seq colname) "Unknown" "" nil nil nil filename nil))
 
 (defun find-ucol (colname filename)
@@ -193,10 +197,10 @@ Currently, these are the LEX and NET files."
   "Returns umls-file structure for a filename"  
   (find-if #'(lambda (f) (string-equal filename (fil f))) *umls-files*))
 
-(defun find-ucols-for-filename (filename)
+(defun find-ucols-for-ufile (ufile)
   "Returns list of umls-cols for a file structure"
-  (loop for colname in (fields (find-ufile filename))
-	collect (find-ucol colname filename)))
+  (loop for colname in (fields ufile)
+	collect (find-ucol colname (fil ufile))))
 
 (defun umls-field-string-to-list (fmt)
   "Converts a comma delimited list of fields into a list of field names. Will
@@ -204,7 +208,7 @@ append a unique number (starting at 2) onto a column name that is repeated in th
   (let ((col-counts (make-hash-table :test 'equal)))
     (loop for colname in (delimited-string-to-list (escape-column-name fmt) #\,)
 	  collect
-	  (multiple-value-bind (value found) (gethash col col-counts)
+	  (multiple-value-bind (value found) (gethash colname col-counts)
 	    (cond
 	      (found
 		(incf (gethash colname col-counts))
@@ -216,7 +220,7 @@ append a unique number (starting at 2) onto a column name that is repeated in th
 (defun make-ufile (fil des table cls rws bts fields)
   (let ((ufile (make-instance 'ufile :fil fil :des des :table table :cls cls
 			      :rws rws :bts bts :fields fields)))
-    (setf (ucols ufile) (find-ucols-for-filename fil))
+    (setf (ucols ufile) (find-ucols-for-ufile ufile))
     ufile))
 
 (defun datatype-for-colname (colname)