Comment 4 for bug 1395242

Revision history for this message
Mark Cox (markcox80) wrote :

Sure. The patch is attached.

I have added a test to tests/grovel.lisp. The test uses the constant
UINT_LEAST32_MAX from stdint.h rather than INADDR_BROADCAST as used in
the bug report. INADDR_BROADCAST requires platform specific header
files, making the test case messy to implement. The header file
stdint.h is included in grovel/common.h and thus is not required to be
included.

The patch to (DEFINE-GROVEL-SYNTAX CONSTANTENUM) and
(DEFINE-GROVEL-SYNTAX CENUM) uses the same style found in
(DEFINE-GROVEL-SYNTAX BITFIELD).