X-Git-Url: https://git.distorted.org.uk/~mdw/sod/blobdiff_plain/be01e762d1fd40a88b26f4bdd40f8f9449d79e0a..00d59354c311fb28730b7c9b117b0d91aac092cc:/src/pset-proto.lisp

diff --git a/src/pset-proto.lisp b/src/pset-proto.lisp
index 61793b5..f03ec51 100644
--- a/src/pset-proto.lisp
+++ b/src/pset-proto.lisp
@@ -43,9 +43,9 @@
 	     (:predicate propertyp)
 	     (:conc-name p-)
 	     (:constructor %make-property
-			   (name value
-			    &key type location seenp
-			    &aux (key (property-key name)) (%type type))))
+		 (name value
+		  &key type location seenp
+		  &aux (key (property-key name)) (%type type))))
   "A simple structure for holding a property in a property set.
 
    The main useful feature is the ability to tick off properties which have
@@ -66,14 +66,8 @@
 (export 'decode-property)
 (defgeneric decode-property (raw)
   (:documentation "Decode a RAW value into a TYPE, VALUE pair.")
-  (:method ((raw symbol)) (values :symbol raw))
-  (:method ((raw integer)) (values :int raw))
-  (:method ((raw string)) (values :string raw))
-  (:method ((raw character)) (values :char raw))
   (:method ((raw property)) (values (p-type raw) (p-value raw)))
-  (:method ((raw cons)) (values (car raw) (cdr raw)))
-  (:method ((raw function)) (values :func raw))
-  (:method ((raw c-type)) (values :type raw)))
+  (:method ((raw cons)) (values (car raw) (cdr raw))))
 
 (export 'make-property)
 (defun make-property (name raw-value &key type location seenp)
@@ -86,52 +80,6 @@
 		    :location (file-location location)
 		    :seenp seenp)))
 
-(defun string-to-symbol
-    (string &key (package *package*) (swap-case t) (swap-hyphen t))
-  "Convert STRING to a symbol in PACKAGE.
-
-   Parse off a `PACKAGE:' prefix from STRING if necessary, to identify the
-   package; PACKAGE is used if there isn't a prefix.  A doubled colon allows
-   access to internal symbols, and will intern if necessary.  Note that
-   escape characters are /not/ processed; don't put colons in package names
-   if you want to use them from SOD property sets.
-
-   The portions of the string are modified by `frob-identifier'; the
-   arguments SWAP-CASE and SWAP-HYPHEN are passed to `frob-identifier' to
-   control this process."
-
-  (let* ((length (length string))
-	 (colon (position #\: string)))
-    (multiple-value-bind (start internalp)
-	(cond ((not colon) (values 0 t))
-	      ((and (< (1+ colon) length)
-		    (char= (char string (1+ colon)) #\:))
-	       (values (+ colon 2) t))
-	      (t
-	       (values (1+ colon) nil)))
-      (when colon
-	(let* ((package-name (if (zerop colon) "KEYWORD"
-				 (frob-identifier (subseq string 0 colon)
-						  :swap-case swap-case
-						  :swap-hyphen swap-hyphen)))
-	       (found (find-package package-name)))
-	  (unless found
-	    (error "Unknown package `~A'" package-name))
-	  (setf package found)))
-      (let ((name (frob-identifier (subseq string start)
-				   :swap-case swap-case
-				   :swap-hyphen swap-hyphen)))
-	(multiple-value-bind (symbol status)
-	    (funcall (if internalp #'intern #'find-symbol) name package)
-	  (cond ((or internalp (eq status :external))
-		 symbol)
-		((not status)
-		 (error "Symbol `~A' not found in package `~A'"
-			name (package-name package)))
-		(t
-		 (error "Symbol `~A' not external in package `~A'"
-			name (package-name package)))))))))
-
 (export 'coerce-property-value)
 (defgeneric coerce-property-value (value type wanted)
   (:documentation
@@ -145,9 +93,10 @@
   ;; say it didn't work.
   (:method (value type wanted)
     (if (eql type wanted) value
-	(error "Incorrect type: expected ~A but found ~A" wanted type)))
+	(error "Incorrect type: expected ~(~A~) but found ~(~A~)"
+	       wanted type)))
 
-  ;; If the caller asks for type T then give him the raw thing.
+  ;; If the caller asks for type T then give them the raw thing.
   (:method (value type (wanted (eql t)))
     (declare (ignore type))
     value))
@@ -306,6 +255,7 @@
 ;;;--------------------------------------------------------------------------
 ;;; Utility macros.
 
+(export 'default-slot-from-property)
 (defmacro default-slot-from-property
     ((instance slot &optional (slot-names t))
      (pset property type