From: Andy Wingo <wingo@pobox.com>
To: guile-devel@gnu.org
Subject: Re: slowness in guile 1.8
Date: Sat, 26 May 2007 12:49:54 +0200 [thread overview]
Message-ID: <1180176594.4388.29.camel@localhost.localdomain> (raw)
In-Reply-To: <87tzu0pybk.fsf@chbouib.org>
Hey Ludovic,
Thanks for looking at this!
On Fri, 2007-05-25 at 20:12 +0200, Ludovic Courtès wrote:
> Alas, it breaks the following test in `syntax.test':
I'm not sure what "it" is in this case; I assume you mean the fix to
module-make-local-var!.
> (pass-if "binding is created before expression is evaluated"
> (= (eval '(begin
> (define foo
> (begin
> (set! foo 1)
> (+ foo 1)))
> foo)
> (interaction-environment))
> 2))
>
> This test case illustrates the fact that _internal_ defines are
> equivalent to `letrec' (Section 5.2.2); top-level defines should behave
> similarly for new variables (Section 5.2.1).
I don't know what you are trying to say here; top-level defines do not
"behave similarly" to letrec. R5RS says in section 5.2.1:
If <variable> is not bound, however, then the definition will
bind <variable> to a new location before performing the
assignment, whereas it would be an error to perform a `set!' on
an unbound variable.
The new variable should be created before the assignment, but _not
necessarily before evaluation of the rhs_.
> For top-level defines as in `(define round round)', the rule is that
> `define' is equivalent to `set!' when the variable is already bound
> (Section 5.2.1). This justifies the change made by Marius to
> `module-make-local-var!' (above).
Only if foo is already bound, in the case you gave, is the test
syntactically valid.
I think the test is bogus. MzScheme, Gambit, Scheme48, and Guile 1.6
think so too. You have snatched defeat from the jaws of victory ;-)
Suggest removing the test.
Cheers,
Andy.
--
http://wingolog.org/
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel
next prev parent reply other threads:[~2007-05-26 10:49 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-25 16:33 slowness in guile 1.8 Andy Wingo
2007-05-25 18:12 ` Ludovic Courtès
2007-05-26 10:49 ` Andy Wingo [this message]
2007-05-26 10:57 ` Andy Wingo
2007-05-26 13:15 ` Ludovic Courtès
2007-05-26 14:45 ` Ludovic Courtès
2007-05-26 15:39 ` Andy Wingo
2007-06-13 22:24 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1180176594.4388.29.camel@localhost.localdomain \
--to=wingo@pobox.com \
--cc=guile-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).