fix idiom for setting stdio stream orientation to wide
authorRich Felker <dalias@aerifal.cx>
Sat, 13 Jun 2015 05:17:16 +0000 (05:17 +0000)
committerRich Felker <dalias@aerifal.cx>
Sat, 13 Jun 2015 05:17:16 +0000 (05:17 +0000)
commit536c6d5a4205e2a3f161f2983ce1e0ac3082187d
treea45cf47d93d62585cd417b83bd49100edf419222
parentf8f565df467c13248104223f99abf7f37cef7584
fix idiom for setting stdio stream orientation to wide

the old idiom, f->mode |= f->mode+1, was adapted from the idiom for
setting byte orientation, f->mode |= f->mode-1, but the adaptation was
incorrect. unless the stream was alreasdy set byte-oriented, this code
incremented f->mode each time it was executed, which would eventually
lead to overflow. it could be fixed by changing it to f->mode |= 1,
but upcoming changes will require slightly more work at the time of
wide orientation, so it makes sense to just call fwide. as an
optimization in the single-character functions, fwide is only called
if the stream is not already wide-oriented.
src/stdio/fgetwc.c
src/stdio/fputwc.c
src/stdio/fputws.c
src/stdio/ungetwc.c
src/stdio/vfwprintf.c
src/stdio/vfwscanf.c