[Git][cmucl/cmucl][issue-143-ansi-compliance-failure-listen-extra-argument] adds argument checking for lisp-streams

[Jon Boone (@jboone)]

Jon Boone pushed to branch issue-143-ansi-compliance-failure-listen-extra-argument at cmucl / cmucl


Commits:
41bbfcd6 by Jon Boone at 2023-05-02T02:14:01-04:00
adds argument checking for lisp-streams

- - - - -


1 changed file:

- src/code/stream.lisp


Changes:

=====================================
src/code/stream.lisp
=====================================
@@ -604,7 +604,7 @@
 	   :skipped-char-form ()
 	   :eof-detected-form (eof-or-lose stream eof-errorp eof-value))))))
 
-(defun listen (&optional (stream *standard-input*) (width 1))
+(defun listen (&optional (stream *standard-input* stream-p) (width 1 width-p))
   "Returns T if a character is available on the given Stream."
   (declare (type streamlike stream))
   (let ((stream (in-synonym-of stream)))
@@ -612,9 +612,26 @@
       ;; simple-stream
       (stream::%listen stream width)
       ;; lisp-stream
-      (or (/= (the fixnum (lisp-stream-in-index stream)) in-buffer-length)
-	  ;; Test for t explicitly since misc methods return :eof sometimes.
-	  (eq (funcall (lisp-stream-misc stream) stream :listen) t))
+      (let ((error-type 'simple-program-error)
+            (function-name 'listen)
+            (format-control ())
+            (format-arguments ()))
+        (if width-p
+          ;; since width provided, two possible cases:
+          (progn
+            (if stream-p
+              ;; stream also provided, so too many arguments
+              (setf format-control (intl:gettext "Invalid number of arguments: ~S")
+                    format-arguments (list 3))
+              ;; stream init-form used, so invalid argument
+              (setf format-control (intl:gettext "Invalid argument: ~D")
+                    format-arguments (list 'width)))
+            (error error-type :function-name function-name
+                   :format-control format-control :format-arguments format-arguments))
+          ;; width not provided, so return expected value
+          (or (/= (the fixnum (lisp-stream-in-index stream)) in-buffer-length)
+            ;; Test for t explicitly since misc methods return :eof sometimes.
+            (eq (funcall (lisp-stream-misc stream) stream :listen) t))))
       ;; fundamental-stream
       (stream-listen stream))))
 



View it on GitLab: https://gitlab.common-lisp.net/cmucl/cmucl/-/commit/41bbfcd64ccf2d21f9f149849494ee7417dabde6

-- 
View it on GitLab: https://gitlab.common-lisp.net/cmucl/cmucl/-/commit/41bbfcd64ccf2d21f9f149849494ee7417dabde6
You're receiving this email because of your account on gitlab.common-lisp.net.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.common-lisp.net/pipermail/cmucl-cvs/attachments/20230502/168d91e6/attachment-0001.html>