This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Fwd: [PATCH v2 3/5] Don't install libio.h or _G_config.h.
[forwarding accidental off-list reply]
On Tue, Feb 6, 2018 at 8:50 AM, Adhemerval Zanella
<adhemerval.zanella@linaro.org> wrote:
> On 05/02/2018 21:34, Zack Weinberg wrote:
>> In this patch, we stop installing libio.h and _G_config.h for public
>> use; necessarily, this means the public stdio.h stops including
>> libio.h.
>
> It looks good to me, with just a couple of nits below (which do not
> block the patch itself).
I can't promise to do any follow-up work anytime soon.
>> -#ifndef BUFSIZ
>> -# define BUFSIZ _IO_BUFSIZ
>> -#endif
>> +#define BUFSIZ 8192
>
> I think a possible subsqueuent cleanup is to remove _IO_BUFSIZE utilization
> within glibc as well.
>
>> +/* The value returned by fgetc and similar functions to indicate the
>> + end of the file. */
>> +#define EOF (-1)
>
> EOF is being define on both libio/libioP.h and libio/stdio.h. Shouldn't we
> cleanup the libioP.h one?
These are good suggestions for further cleanups but I'm not going to
fold them into this patchset. I don't think there's any danger of
them getting forgotten about; whoever next looks at libio(P).h is
going to be looking for cleanups.
zw