Re: [PATCH 09/17] lib: add fileutils function collection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 9, 2012 at 13:20, Karel Zak <kzak@xxxxxxxxxx> wrote:
> On Fri, Mar 09, 2012 at 12:48:55PM +0100, Davidlohr Bueso wrote:
>> I don't see how streams are more complete than descriptors - and no,
>> every call to mkstemp *does not* require a call to fdopen, that's
>> completely up to the user's use case.
>
>  Add two functions :-)
>
>    int xmkstemp()
>    FILE *xfmkstemp()
>
>  the second function might be an inline function...

Hi Karel & Davidlohr,

Done the suggested way.

https://github.com/kerolasa/lelux-utiliteetit/commit/1d2594dde3de6bf4e23a284c08dc4c8cb7c86a6f

BTW I noticed mistake in earlier change, which made file descriptors
to be preferred way of changing permissions etc. The issue was that fd
was referred after closure. That is now fixed in commit below.

https://github.com/kerolasa/lelux-utiliteetit/commit/58681bfe79ed03d2b3c39f81b14cdb4723a51b63

Assuming no more comments arrive the week 9 patches might be ready for
final maintainer review hopefully resulting to merge.

-- 
   Sami Kerola
   http://www.iki.fi/kerolasa/
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux