[PATCH] Add getmntent_r
Yaakov (Cygwin/X)
yselkowitz@users.sourceforge.net
Wed Jul 18 20:24:00 GMT 2012
On 2012-07-18 06:17, Corinna Vinschen wrote:
> On Jul 18 05:21, Yaakov (Cygwin/X) wrote:
>> On 2012-06-06 02:33, Corinna Vinschen wrote:
>>> In case of Cygwin this is not needed since we don't read from the file
>>> but from the internal datastructure. There's no reason to create
>>> garbage in buf just because this is by chance the layout the buffer gets
>>> when operating under Linux.
>>>
>>> The *important* thing is that buf contains the strings the members of
>>> mntbuf points to.
>>
>> OK, revised patch attached.
>
> Thanks. Applied with a tweak: It's really not necessary at all to
> create strings for mnt_freq and mnt_passno in buf. Just copy them
> over from mnt to mntbuf and be done with it.
In that case, we don't need opts_len, and AFAICS it will introduce a
warning with GCC 4.6 [-Wunused-but-set-variable]. OK to remove?
Yaakov
More information about the Cygwin-patches
mailing list