Guenter Knauf wrote:
> ftp_commands.c: In function ‘common_list’:
> 
> ftp_commands.c:694: warning: suggest parentheses around && within ||

As we worked out, the distinction made no effective difference, but...

> ftp_protocol.c: In function ‘ftp_read_line’:
> ftp_protocol.c:244: warning: comparison is always false due to limited
> range of data type
> ftp_protocol.c:246: warning: comparison is always false due to limited
> range of data type
> ftp_protocol.c:246: warning: comparison is always true due to limited
> range of data type
> ftp_protocol.c:255: warning: comparison is always false due to limited
> range of data type
> ftp_protocol.c:258: warning: comparison is always false due to limited
> range of data type
> ftp_protocol.c:258: warning: comparison is always true due to limited
> range of data type
> 
> we should fix these first, and re-roll ...

that sounds right; this code was faulty on platforms where '\xff' != 0xff.

Reply via email to