Re: [Findutils-patches] patch for #20662 memory leak

[James Youngman]

On 8/5/07, Eric Blake <address@hidden> wrote:
>
> OK to commit?  Should I port the entire patch back to the branch,
> including the -nowarn and testsuite changes; or just the memory leak fix?

Great patch; thanks!    Yes, please commit.  I also think it is
worthwhile backporting the whole thing to 4.2.x.

Just one tiny nit...

> +  if (strchr(arg, '/') && options.warnings)

Reversing this test will avoid a tiny performance hit for long arguments.

Thanks!
James.