[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] Two small patches
From: |
Evan Hanson |
Subject: |
Re: [Chicken-hackers] [PATCH] Two small patches |
Date: |
Thu, 5 May 2016 13:18:55 +1200 |
Hi Lemon,
The first patch looks fine to me. Here's a signoff.
I don't care for the second patch, personally. The current message makes
sense in context when warnings aren't disabled, and if we're being
pedantic the symbols aren't really undefined either, but unbound.
Something like "unbound variables in module" might be better, but in
general if you need verbose errors when building core you should just
set DEBUGBUILD.
Cheers,
Evan