r11481: make vff macro more efficient

[umlisp.git] / parse-rrf.lisp

diff --git a/parse-rrf.lisp b/parse-rrf.lisp
index e8e919de245a0e34da419ac546bf35b7072cfc9c..90a11fcc5a63e3e34843b68bd7ce6fd79f532993 100644 (file)
--- a/parse-rrf.lisp
+++ b/parse-rrf.lisp
@@ -187,14 +187,24 @@ used in the United States. We create a new scale (SRLUS) where SRL to SRLUS mapp
   '(("KCON" "SELECT CUI,STR FROM MRCONSO WHERE STT='PF' AND TS='P' AND ISPREF='Y' AND LAT='ENG'"))
   "Custom tables to create")
 
+(defvar *vff-position-hash* (make-hash-table :size 100 :test 'eq))
+
+
 (defmacro vff (filename fieldname record)
   (let ((pos (gensym "POS-"))
+        (found (gensym "FOUND-"))
         (key (kmrcl:ensure-keyword (concatenate 'string filename "^" fieldname))))
-    `(let ((,pos (position-field-file ,filename ,fieldname ,key)))
-      (unless ,pos
-        (error "Did not find fieldname ~A in filename ~A." ,fieldname ,filename))
-      (locally (declare (type (integer 0 100000) ,pos))
-        (nth ,pos ,record)))))
+    `(locally (declare (optimize (speed 3) (size 0) (safety 0)))
+      (multiple-value-bind (,pos ,found) (gethash ,key *vff-position-hash*)
+        (if ,pos
+            (locally (declare (type (integer 0 100000) ,pos))
+              (nth ,pos ,record))
+            (let ((,pos (position-field-file ,filename ,fieldname)))
+              (unless ,pos
+                (error "Did not find fieldname ~A in filename ~A." ,fieldname ,filename))
+              (locally (declare (type (integer 0 100000) ,pos))
+                (setf (gethash ,key *vff-position-hash*) ,pos)
+                (nth ,pos ,record))))))))
 
 (defparameter +custom-cols+
     '(#+nil ("MRCONSO.RRF" "KPFSTR" "TEXT"