Source-Changes archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src
In article <20080313162731.GE1065%britannica.bec.de@localhost>,
Joerg Sonnenberger <joerg%britannica.bec.de@localhost> wrote:
>On Thu, Mar 13, 2008 at 03:40:01PM +0000, Christos Zoulas wrote:
>> Log Message:
>> Since _file is a short, check that the fd fits in it, otherwise bail with
>> EMFILE. We treat _file as an unsigned short to double our range, with a
>> special case for -1 (closed). Make a note of what we should do about stdio
>> if we ever bump libc. We could change _file in the future compatibly to an
>> int by putting it in the extension space but for now we don't bother.
>
>I don't think that it is the correct approach. The change still broke
>fileno usage as the macro will return a short value and that isn't
>usable. If breaking fileno users is not considered a problem the best
>fix is to actually just put the real _file field into the extended area
>and put -1 into _file iff the real _file is -1 or larger than 32767.
How? The macro now returns an int value...
christos
Home |
Main Index |
Thread Index |
Old Index