unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
* bug#22767: Thread-sleep! doesn't sleep
@ 2016-02-22 11:37 Takashi Kato
  2016-08-07 21:32 ` Andy Wingo
  0 siblings, 1 reply; 2+ messages in thread
From: Takashi Kato @ 2016-02-22 11:37 UTC (permalink / raw)
  To: 22767

Hi there,

I think I've found a bug of thread-sleep! defined in SRFI-18 library.
The file module/srfi/srfi-18.scm line 233 subtract current time from
given timeout argument but this results negative number most of the
case. I think this line should simply return timeout as it is.

Cheers,

-- 
_/_/
Takashi Kato
E-mail: ktakashi19@gmail.com





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

* bug#22767: Thread-sleep! doesn't sleep
  2016-02-22 11:37 bug#22767: Thread-sleep! doesn't sleep Takashi Kato
@ 2016-08-07 21:32 ` Andy Wingo
  0 siblings, 0 replies; 2+ messages in thread
From: Andy Wingo @ 2016-08-07 21:32 UTC (permalink / raw)
  To: Takashi Kato; +Cc: 22767

Hi Takashi,

Thank you for the report.

On Mon 22 Feb 2016 12:37, Takashi Kato <ktakashi19@gmail.com> writes:

> I think I've found a bug of thread-sleep! defined in SRFI-18 library.
> The file module/srfi/srfi-18.scm line 233 subtract current time from
> given timeout argument but this results negative number most of the
> case. I think this line should simply return timeout as it is.

You are right.  Guile's documentation indicates that the timeout is an
absolute time as a SRFI-18 time object, but the SRFI itself says that
timeouts are either:

   * a time object represents an absolute point in time

   * an exact or inexact real number represents a relative time in
     seconds from the moment the primitive was called

So I guess when passed a number, that's not to be interpreted as an
absolute time from the epoch, but rather a relative timeout.  We need to
update our documentation and tests, it seems, and issue a prominent NEWS
entry...

Andy





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

end of thread, other threads:[~2016-08-07 21:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-22 11:37 bug#22767: Thread-sleep! doesn't sleep Takashi Kato
2016-08-07 21:32 ` Andy Wingo

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).