[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Guile-commits] GNU Guile branch, stable-2.0, updated. v2.0.5-68-gda

From: Andy Wingo
Subject: Re: [Guile-commits] GNU Guile branch, stable-2.0, updated. v2.0.5-68-gdab48cc
Date: Tue, 06 Mar 2012 21:18:52 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux)

On Tue 06 Mar 2012 18:13, address@hidden (Ludovic Courtès) writes:

> "Andy Wingo" <address@hidden> skribis:
>> commit dab48cc567f931b275ad647db1e47135b63c6675
>> Author: Andy Wingo <address@hidden>
>> Date:   Fri Mar 2 17:46:28 2012 +0100
>>     tweaks to -Wformat's gettext detection
>>     * module/language/tree-il/analyze.scm (proc-ref?): Change to use less
>>       false-if-exception and more variable-bound?.  If a variable is present
>>       in the local module but not bound, assume that it is gettext if it has
>>       the right name.  This is to allow for (define _ gettext).
> What’s the functional change?

The previous behavior depended on the side effect of the expander
producing an unbound variable in the current module.  This behavior is
not present in master.  I changed the test case so as to do something
that would work in master.

> Could you reinstate "non-literal format string with forward
> declaration", which tests something different as the name implies (see
> d3160473)?

Sure.  First, though, would you mind discussing the desired behavior a
bit more?

I think the "forward declaration" test was not quite right, because it
would not work with a local (define _ gettext).  That's why I changed
the test from checking that a warning was issued on a local (define _
...) to change that a warning was not issued on that same case.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]