unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
* bug#62690: Guile 3.0.9 (read-u8) defaults to current-output-port
@ 2023-04-05 20:38 Rui Zhang
  2023-07-16 20:31 ` Ludovic Courtès
  0 siblings, 1 reply; 2+ messages in thread
From: Rui Zhang @ 2023-04-05 20:38 UTC (permalink / raw)
  To: 62690

Reproduction:

scheme@(guile-user)> (import (scheme base))
scheme@(guile-user)> (read-u8)
ice-9/boot-9.scm:1685:16: In procedure raise-exception:
In procedure get-u8: Wrong type argument in position 1 (expecting open input port): #<output: file /dev/pts/9>

Expected: (read-u8) should read a byte from the input port, not the output port.

In scheme/base.scm:

(define* (read-u8 #:optional (port (current-output-port)))
  (get-u8 port))

I think this is a typo, where current-output-port should be current-input-port?

-Rui




^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#62690: Guile 3.0.9 (read-u8) defaults to current-output-port
  2023-04-05 20:38 bug#62690: Guile 3.0.9 (read-u8) defaults to current-output-port Rui Zhang
@ 2023-07-16 20:31 ` Ludovic Courtès
  0 siblings, 0 replies; 2+ messages in thread
From: Ludovic Courtès @ 2023-07-16 20:31 UTC (permalink / raw)
  To: Rui Zhang; +Cc: 62690-done

Hi,

Rui Zhang <zrui16@hotmail.com> skribis:

> In scheme/base.scm:
>
> (define* (read-u8 #:optional (port (current-output-port)))
>   (get-u8 port))
>
> I think this is a typo, where current-output-port should be current-input-port?

Oops, fixed, thanks!

Ludo’.





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-07-16 20:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-05 20:38 bug#62690: Guile 3.0.9 (read-u8) defaults to current-output-port Rui Zhang
2023-07-16 20:31 ` Ludovic Courtès

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).