Re: [Patch] Add dirent.d_type support to Cygwin 1.7 ?

[Jim Meyering]

Jim Meyering <address@hidden> wrote:

> Jim Meyering <address@hidden> wrote:
>> "James Youngman" <address@hidden> wrote:
>>> As far as I can see, the variable type is assigned in the function
>>> above, but never used.   Did you mean to use "type" rather than
>>> "dtype" in the expression which assigns to st->st_mode?
>>
>> Thanks again.
>> Here's the patch I'll push as soon as you say ok:
>
> I hadn't tested enough after all.
> That didn't work either.  But this does.
> With *it*, all 1133 of find's tests really do pass:
>
>>From fb3d1318c0ea7f675b0702897c35f83198efdae8 Mon Sep 17 00:00:00 2001
> From: Jim Meyering <address@hidden>
> Date: Sat, 29 Nov 2008 16:08:09 +0100
> Subject: [PATCH] fts: fix a thinko
>
> * lib/fts.c (s_ifmt_shift_bits): Remove function.  Not needed after all.
> (set_stat_type): Return S_IF*-valued "type" directly.
> Prompted by James Youngman's spotting a related bug.
> Confirmed by further testing through find.

I've just pushed that one.