On Sat, Nov 29, 2008 at 3:53 PM, James Youngman <[EMAIL PROTECTED]> wrote: > On Sat, Nov 29, 2008 at 3:46 PM, Jim Meyering <[EMAIL PROTECTED]> wrote: >> Jim Meyering <[EMAIL PROTECTED]> wrote: >>> "James Youngman" <[EMAIL PROTECTED]> 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: > > Looks good to me, though I did no local testing. > > I will apply your find patch at some point this weekend, I hope.
I've applied the patch to the CVS trunk for findutils; it will appear in findutils 4.5.3, whenever that is released. Thanks! James.
