This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Mon, Oct 26, 2015 at 1:06 PM, Rich Felker <dalias@libc.org> wrote: > This highly pessimizes short reads/writes, e.g. fwrite(&c,1,1,f), by > introducing a div operation. The obvious intent of the original code > was to avoid this. Thanks for comments, everyone. Revised patch addresses all of them (I believe). Not sure whether the name or location of the saturating multiplication is acceptable though. Suggestions welcome. 2015-10-26 Paul Pluzhnikov <ppluzhnikov@google.com> [BZ #19165] * libio/libioP.h (_IO_saturating_umull): New. * libio/iofread.c (_IO_fread): Use it. * ibio/iofread_u.c (__fread_unlocked): Likewise. * libio/iofwrite.c (_IO_fwrite): Error on overflow. * libio/iofwrite_u.c (fwrite_unlocked): Likewise. -- Paul Pluzhnikov
Attachment:
bz19165-20151026.txt
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |