Yaakov (Cygwin/X | 21 Feb 2012 23:44
Picon
Gravatar

[PATCH] Add scandirat(3)

scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
for addition to POSIX.1[2].  Patch attached.

Yaakov

[1] http://sourceware.org/git/?p=glibc.git;a=blob_plain;f=NEWS
[2] http://article.gmane.org/gmane.linux.man/2419

Attachment (cygwin-scandirat.patch): text/x-patch, 4250 bytes
Eric Blake | 22 Feb 2012 00:03
Picon
Favicon
Gravatar

Re: [PATCH] Add scandirat(3)

On 02/21/2012 03:44 PM, Yaakov (Cygwin/X) wrote:
> scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
> for addition to POSIX.1[2].  Patch attached.

I haven't yet seen anyone propose it for POSIX, but it would indeed be a
welcome addition there.

Also it would be a welcome addition to have pathconfat(), although this
hasn't yet happened on the Linux side of things, let alone any POSIX
proposal.

At any rate, +1 for having this in cygwin.

--

-- 
Eric Blake   eblake <at> redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Christopher Faylor | 22 Feb 2012 02:30
Favicon

Re: [PATCH] Add scandirat(3)

On Tue, Feb 21, 2012 at 04:44:58PM -0600, Yaakov (Cygwin/X) wrote:
>scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
>for addition to POSIX.1[2].  Patch attached.
>
>
>Yaakov
>
>[1] http://sourceware.org/git/?p=glibc.git;a=blob_plain;f=NEWS
>[2] http://article.gmane.org/gmane.linux.man/2419
>

>2012-02-??  Yaakov Selkowitz <yselkowitz <at> ...>
>
>	* cygwin.din (scandirat): Export.
>	* posix.sgml (std-gnu): Add scandirat.
>	* syscalls.cc (scandirat): New function.
>	* include/cygwin/version.h (CYGWIN_VERSION_API_MINOR): Bump.
>	* include/sys/dirent.h (scandirat): Declare.

Looks good.  Please apply.

Thanks, as always.

cgf

Corinna Vinschen | 24 Feb 2012 14:09
Favicon

Re: [PATCH] Add scandirat(3)

On Feb 21 16:44, Yaakov (Cygwin/X) wrote:
> scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
> for addition to POSIX.1[2].  Patch attached.
> 
> 
> Yaakov
> 
> [1] http://sourceware.org/git/?p=glibc.git;a=blob_plain;f=NEWS
> [2] http://article.gmane.org/gmane.linux.man/2419
> [...]

Is it just me or do others have the same feeling that the function name
is a bit... unfortunate.  When I read this patch proposal the first time
I thought something like "scan-di-rat?  What the heck does *that* mean?

A couple of days later I still tend to pronounce it "scan-di-rat" rather
than "scan-dir-at", and I have serious doubts that I will adapt...

Corinna

Warren Young | 24 Feb 2012 15:01
Favicon

Re: [PATCH] Add scandirat(3)

On 2/24/2012 6:09 AM, Corinna Vinschen wrote:
> On Feb 21 16:44, Yaakov (Cygwin/X) wrote:
>> scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
>> for addition to POSIX.1[2].  Patch attached.
>>
> I thought something like "scan-di-rat?  What the heck does *that* mean?

int scandirat(RAT* pRat, IWiaDataTransfer* pScanner,
     int dpi, int w, int h, HBITMAP hb);

Dave Korn | 27 Feb 2012 16:46
Picon

Re: [PATCH] Add scandirat(3)

On 24/02/2012 13:09, Corinna Vinschen wrote:
> On Feb 21 16:44, Yaakov (Cygwin/X) wrote:
>> scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
>> for addition to POSIX.1[2].  Patch attached.
>>
>>
>> Yaakov
>>
>> [1] http://sourceware.org/git/?p=glibc.git;a=blob_plain;f=NEWS
>> [2] http://article.gmane.org/gmane.linux.man/2419
>> [...]
> 
> Is it just me or do others have the same feeling that the function name
> is a bit... unfortunate.  When I read this patch proposal the first time
> I thought something like "scan-di-rat?  What the heck does *that* mean?
> 
> A couple of days later I still tend to pronounce it "scan-di-rat" rather
> than "scan-dir-at", and I have serious doubts that I will adapt...

  'slike a sugar mouse, but bigger.  'scandirat.

    cheers,
      DaveK

Corinna Vinschen | 27 Feb 2012 18:08
Favicon

Re: [PATCH] Add scandirat(3)

On Feb 27 15:46, Dave Korn wrote:
> On 24/02/2012 13:09, Corinna Vinschen wrote:
> > On Feb 21 16:44, Yaakov (Cygwin/X) wrote:
> >> scandirat(3) was added in glibc-2.15[1] and has supposedly been proposed
> >> for addition to POSIX.1[2].  Patch attached.
> >>
> >>
> >> Yaakov
> >>
> >> [1] http://sourceware.org/git/?p=glibc.git;a=blob_plain;f=NEWS
> >> [2] http://article.gmane.org/gmane.linux.man/2419
> >> [...]
> > 
> > Is it just me or do others have the same feeling that the function name
> > is a bit... unfortunate.  When I read this patch proposal the first time
> > I thought something like "scan-di-rat?  What the heck does *that* mean?
> > 
> > A couple of days later I still tend to pronounce it "scan-di-rat" rather
> > than "scan-dir-at", and I have serious doubts that I will adapt...
> 
>   'slike a sugar mouse, but bigger.  'scandirat.

I likes candy...

Corinna


Gmane