[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: build-lexical-var vs. -Wunused-variable
From: |
Andy Wingo |
Subject: |
Re: build-lexical-var vs. -Wunused-variable |
Date: |
Sun, 19 Feb 2012 22:08:26 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) |
On Fri 17 Feb 2012 12:00, address@hidden (Ludovic Courtès) writes:
> Commit f9685f437312ea790981373ddc375b2a26ba9c4f changes
> ‘build-lexical-var’ like this:
>
> (define-syntax-rule (build-lexical-var src id)
> - (gensym (string-append (symbol->string id) " ")))
> + (gensym (string-append (symbol->string id) "-")))
>
> This breaks a heuristic used in the ‘unused-variable’ pass, which is
> that identifiers containing white space are likely introduced by a macro
> and can be ignored in unused-variable reports (see ‘gensym?’ in
> ‘tree-il/analyze.scm’).
>
> OK to change it back?
How about we use `$' instead? I understand that's how Chez does it.
I'd really rather something that is printable without the #{#} syntax.
Andy
--
http://wingolog.org/