ludo@gnu.org (Ludovic Courtès) writes: > Greg Troxel writes: > >> #define NULL (void *)0 > > This is under-parenthesized! > > Two actions to be taken: > > 1. Report a bug to your favorite libc maintainers. ;-) I think that's a legal value for NULL per C99. Interestingly it has not apparently caused other trouble. We'll see what others say, and I can see the point that having it parenthesized is defensive even if it's legal as is. > 2. Ask libunistring folks at > http://savannah.gnu.org/projects/libunistring/ to replace > "sizeof NULL" by "sizeof (NULL)" in `test-locale.c'. I did, and this has been committed to their repo for gnulib.