Re: Warnings to be fixed

[Bruno Haible]

Reuben Thomas wrote:
> gl_anylinked_list2.h: In function ‘gl_linked_iterator’:
> gl_anylinked_list2.h:818: warning: ‘result.j’ is used uninitialised in this 
> function
> gl_anylinked_list2.h:818: warning: ‘result.i’ is used uninitialised in this 
> function
> gl_anylinked_list2.h:818: warning: ‘result.count’ is used uninitialised in 
> this function

These warnings are unwarranted: GCC is not doing complete data flow analysis
here.

> printf-args.c: In function ‘printf_fetchargs’:
> printf-args.c:93: warning: signed and unsigned type in conditional expression
> vasnprintf.c: In function ‘vasnprintf’:
> vasnprintf.c:4442: warning: comparison between signed and unsigned

These "signed / unsigned" warnings are more often false alarms than true
alarms. Here both are false alarms. You're welcome to report these warnings
after you verified that they point to real problems, or can be silenced
without adding casts. (Adding a cast is a bad cure, because it makes the
code less maintainable.)

Bruno