unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
* bug#72373: srfi-64: test-assert evaluates test-name multiple times
@ 2024-07-30 19:51 Tomas Volf
  2024-10-01 22:01 ` Taylan Kammer
  0 siblings, 1 reply; 2+ messages in thread
From: Tomas Volf @ 2024-07-30 19:51 UTC (permalink / raw)
  To: 72373

Hello,

I think I found a bug in (srfi srfi-64) module shipped with GNU Guile.

The specification says the following regarding the test-assert:

> Though the test-name is a string literal in the examples, it is an
> expression. It is evaluated only once.

However the test-name is evaluated multiple times:

    (use-modules (srfi srfi-64))
    (test-begin "x")
    (test-assert (pk "t") #t)

Leading to:

    ;;; ("t")

    ;;; ("t")

Have a nice day
Tomas Volf





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

* bug#72373: srfi-64: test-assert evaluates test-name multiple times
  2024-07-30 19:51 bug#72373: srfi-64: test-assert evaluates test-name multiple times Tomas Volf
@ 2024-10-01 22:01 ` Taylan Kammer
  0 siblings, 0 replies; 2+ messages in thread
From: Taylan Kammer @ 2024-10-01 22:01 UTC (permalink / raw)
  To: Tomas Volf, 72373

On 30.07.2024 21:51, Tomas Volf wrote:
> Hello,
>
> I think I found a bug in (srfi srfi-64) module shipped with GNU Guile.
>
> The specification says the following regarding the test-assert:
>
>> Though the test-name is a string literal in the examples, it is an
>> expression. It is evaluated only once.
> However the test-name is evaluated multiple times:
>
>     (use-modules (srfi srfi-64))
>     (test-begin "x")
>     (test-assert (pk "t") #t)
>
> Leading to:
>
>     ;;; ("t")
>
>     ;;; ("t")
>
> Have a nice day
> Tomas Volf
>
>
Exact same story as with the previous bug report regarding `test-approximate`.

This time the affected line is 755. (Need to change `tname` to `name`.)

And likewise, already fixed in my implementation.

- Taylan






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

end of thread, other threads:[~2024-10-01 22:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-30 19:51 bug#72373: srfi-64: test-assert evaluates test-name multiple times Tomas Volf
2024-10-01 22:01 ` Taylan Kammer

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