X-Git-Url: https://git.distorted.org.uk/~mdw/sod/blobdiff_plain/d1c01c331ae3f2994d65b9d41655b0d4f2fcbf3d..a5b9e2bef2e640d841229bd7ec1b69782e6e7e70:/src/module-parse.lisp?ds=sidebyside

diff --git a/src/module-parse.lisp b/src/module-parse.lisp
index 0a3cd28..81a3956 100644
--- a/src/module-parse.lisp
+++ b/src/module-parse.lisp
@@ -226,7 +226,7 @@
     (parse (seq ((make (or (seq ("init") #'make-sod-class-initfrag)
 			   (seq ("teardown") #'make-sod-class-tearfrag)))
 		 (frag (parse-delimited-fragment scanner #\{ #\})))
-	     (funcall make class frag pset scanner)))))
+	     (funcall make class frag pset :location scanner)))))
 
 (define-pluggable-parser class-item initargs (scanner class pset)
   ;; initarg-item ::= `initarg' declspec+ list[init-declarator]
@@ -243,7 +243,9 @@
 			  (make-sod-user-initarg class
 						 (cdr declarator)
 						 (car declarator)
-						 pset init scanner))
+						 pset
+						 :default init
+						 :location scanner))
 			#\,))
 		 (nil (must #\;)))))))
 
@@ -267,7 +269,14 @@
 				  (continue ()
 				    (setf duff t)
 				    (list (find-sod-class "SodObject"))))))
-	      superclasses))
+	      (find-duplicates (lambda (first second)
+				 (declare (ignore second))
+				 (setf duff t)
+				 (cerror* "Class `~A' has duplicate ~
+					   direct superclass `~A'"
+					  name first))
+			       superclasses)
+	      (delete-duplicates superclasses)))
 	   (synthetic-name (or name
 			       (let ((var (synthetic-name)))
 				 (unless pset
@@ -275,7 +284,8 @@
 				 (unless (pset-get pset "nick")
 				   (add-property pset "nick" var :type :id))
 				 var)))
-	   (class (make-sod-class synthetic-name superclasses pset scanner))
+	   (class (make-sod-class synthetic-name superclasses pset
+				  :location scanner))
 	   (nick (sod-class-nickname class)))
 
       (labels ((must-id ()
@@ -306,8 +316,8 @@
 		 ;; Don't allow a method-body here if the message takes a
 		 ;; varargs list, because we don't have a name for the
 		 ;; `va_list' parameter.
-		 (let ((message (make-sod-message class name type
-						  sub-pset scanner)))
+		 (let ((message (make-sod-message class name type sub-pset
+						  :location scanner)))
 		   (if (varargs-message-p message)
 		       (parse #\;)
 		       (parse (or #\; (parse-method-item sub-pset
@@ -323,7 +333,8 @@
 					 scanner #\{ #\}))))
 			  (restart-case
 			      (make-sod-method class sub-nick name type
-					       body sub-pset scanner)
+					       body sub-pset
+					       :location scanner)
 			    (continue () :report "Continue")))))
 
 	       (parse-initializer ()
@@ -343,14 +354,12 @@
 		 (flet ((make-it (name type init)
 			  (restart-case
 			      (progn
-				(make-sod-slot class name type
-					       sub-pset scanner)
+				(make-sod-slot class name type sub-pset
+					       :location scanner)
 				(when init
-				  (make-sod-instance-initializer class
-								 nick name
-								 init
-								 sub-pset
-								 scanner)))
+				  (make-sod-instance-initializer
+				   class nick name init sub-pset
+				   :location scanner)))
 			    (continue () :report "Continue"))))
 		   (parse (and (error ()
 				   (seq ((init (? (parse-initializer))))
@@ -381,7 +390,8 @@
 				       (restart-case
 					   (funcall constructor class
 						    name-a name-b init
-						    sub-pset scanner)
+						    sub-pset
+						    :location scanner)
 					 (continue () :report "Continue")))
 				   (skip-until () #\, #\;))
 				 #\,)