bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: strfile: new module


From: Simon Josefsson
Subject: Re: strfile: new module
Date: Sat, 27 May 2006 12:40:37 +0200
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/22.0.50 (gnu/linux)

Ralf Wildenhues <address@hidden> writes:

> * Ralf Wildenhues wrote on Sat, May 27, 2006 at 12:22:35PM CEST:
>> * Simon Josefsson wrote on Sat, May 27, 2006 at 12:12:59PM CEST:
>> > +  int save_errno = errno;
>> 
>> You are using errno but not including <errno.h>.
>
> ...and realloc but not including <stdlib.h>.
> Please compile your code with lots of warnings enabled.

Fixed in last e-mail too.

> You pass NULL to realloc and possibly to free but are not
> depending on the realloc and free modules.

The free case was fixed in the last e-mail, but isn't realloc supposed
to handle NULL?

> I wonder whether you want to read it in text or in binary
> mode (for Cygwin...).

Both GnuTLS and Shishi wants it in binary mode...  I've fixed this in
my local copy.

Thanks,
Simon




reply via email to

[Prev in Thread] Current Thread [Next in Thread]