X-Git-Url: http://git.kpe.io/?p=clsql.git;a=blobdiff_plain;f=uffi%2Fclsql-uffi.lisp;h=63267022dc7ce8c7710b36edee0c395907ce35d8;hp=a74b75c5fb3ad9606721f679e6dbf9bc6e00bdb6;hb=f1ff5842ce264fbe47e8ef98e33e4aca3b71d863;hpb=50e0a512247d82a6657675a67f5d3caf019a25bb

diff --git a/uffi/clsql-uffi.lisp b/uffi/clsql-uffi.lisp
index a74b75c..6326702 100644
--- a/uffi/clsql-uffi.lisp
+++ b/uffi/clsql-uffi.lisp
@@ -7,7 +7,7 @@
 ;;;; Programmers:   Kevin M. Rosenberg
 ;;;; Date Started:  Mar 2002
 ;;;;
-;;;; $Id: clsql-uffi.lisp,v 1.4 2003/03/29 22:31:43 kevin Exp $
+;;;; $Id$
 ;;;;
 ;;;; This file, part of CLSQL, is Copyright (c) 2002 by Kevin M. Rosenberg
 ;;;;
@@ -16,46 +16,48 @@
 ;;;; (http://opensource.franz.com/preamble.html), also known as the LLGPL.
 ;;;; *************************************************************************
 
-(declaim (optimize (debug 3) (speed 3) (safety 1) (compilation-speed 0)))
-(in-package :clsql-uffi)
+(in-package #:clsql-uffi)
 
 
 (defun canonicalize-type-list (types auto-list)
   "Ensure a field type list meets expectations"
-  (let ((length-types (length types))
-	(new-types '()))
-    (loop for i from 0 below (length auto-list)
-	  do
-	  (if (>= i length-types)
-	      (push t new-types) ;; types is shorted than num-fields
-	      (push
-	       (case (nth i types)
-		 (:int
-		  (case (nth i auto-list)
-		    (:int32
-		     :int32)
-		    (:int64
-		     :int64)
-		    (t
-		     t)))
-		 (:double
-		  (case (nth i auto-list)
-		    (:double
-		     :double)
-		    (t
-		     t)))
-		 (:int32
-		  (if (eq :int32 (nth i auto-list))
-		      :int32
-		    t))
-		 (:int64
-		  (if (eq :int64 (nth i auto-list))
-		      :int64
-		    t))
-		 (t
-		  t))
-	       new-types)))
-    (nreverse new-types)))
+  (declare (optimize (speed 3) (safety 0)))
+  (do ((i 0 (1+ i))
+       (new-types '())
+       (length-types (length types))
+       (length-auto-list (length auto-list)))
+      ((= i length-auto-list)
+       (nreverse new-types))
+    (declare (fixnum length-types length-auto-list i))
+    (if (>= i length-types)
+	(push t new-types) ;; types is shorted than num-fields
+	(push
+	 (case (nth i types)
+	   (:int
+	    (case (nth i auto-list)
+	      (:int32
+	       :int32)
+	      (:int64
+	       :int64)
+	      (t
+	       t)))
+	   (:double
+	    (case (nth i auto-list)
+	      (:double
+	       :double)
+	      (t
+	       t)))
+	   (:int32
+	    (if (eq :int32 (nth i auto-list))
+		:int32
+		t))
+	   (:int64
+	    (if (eq :int64 (nth i auto-list))
+		:int64
+		t))
+	   (t
+	    t))
+	 new-types))))
 
 (uffi:def-function "atoi"
     ((str (* :unsigned-char)))
@@ -78,33 +80,21 @@
 (uffi:def-constant +2^32-1+ (1- +2^32+))
 
 (defmacro make-64-bit-integer (high32 low32)
-  `(+ ,low32 (* ,high32 +2^32+)))
+  `(+ ,low32 (ash ,high32 32)))
 
 (defmacro split-64-bit-integer (int64)
   `(values (ash ,int64 -32) (logand ,int64 +2^32-1+)))
 
-(defconstant +ascii-N-value+ (char-code #\N))
-(defconstant +ascii-U-value+ (char-code #\U))
-
 (uffi:def-type char-ptr-def (* :unsigned-char))
 
-(defun char-ptr-points-to-null (char-ptr)
-  "Returns T if foreign character pointer refers to 'NULL' string. Only called for numeric entries"
-  ;; Uses short cut and returns T if first character is #\N. It should
-  ;; never be non-numeric
-  (declare (type char-ptr-def char-ptr))
-  (let ((char (uffi:ensure-char-character
-	       (uffi:deref-pointer char-ptr :char))))
-    (eql char #\N)))
-    
-(defun convert-raw-field (char-ptr types index)
+(defun convert-raw-field (char-ptr types index &optional length)
+  (declare (optimize (speed 3) (safety 0) (space 0))
+ 	   (type char-ptr-def char-ptr))
   (let ((type (if (listp types)
 		  (nth index types)
 		  types)))
     (cond
-      ((and (or (eq type :double) (eq type :int32) (eq type :int)
-		(eq type :int64))
-	    (char-ptr-points-to-null char-ptr))
+      ((uffi:null-pointer-p char-ptr)
        nil)
       (t
        (case type
@@ -120,5 +110,9 @@
 		  low32
 		  (make-64-bit-integer high32 low32)))))
 	 (t
-	  (uffi:convert-from-foreign-string char-ptr)))))))
-  
+          (if length
+	      (uffi:convert-from-foreign-string char-ptr :locale :none
+                                                :null-terminated-p nil
+                                                :length length)
+            (uffi:convert-from-foreign-string char-ptr :locale :none))))))))
+