chicken-hackers
[Top][All Lists]
Advanced

[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



reply via email to

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