[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Chicken-hackers] [PATCH] Less type smashing
From: |
LemonBoy |
Subject: |
[Chicken-hackers] [PATCH] Less type smashing |
Date: |
Tue, 12 Sep 2017 10:20:24 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
Hello hackers,
after debugging some code I noticed the compiler was smashing the type for some
list I had defined via `define-constant`. That's not needed since the constants
aren't exported nor they are modifiable so here's a patch to make the
scrutinizer aware of that.
One minor flaw I've found is that `define-constant` doesn't play well with the
modules since it does mark a fresh gensym'd variable that's unqualified. That's
bad since the scrutinizer queries the qualified variable name with
`variable-mark` and as a result of this mismatch we do miss the
'##compiler#constant mark.
I don't know what's the best fix for this problem so feel free to chime in :)
Cheers,
LemonBoy
- [Chicken-hackers] [PATCH] Less type smashing,
LemonBoy <=