bug-coreutils
[Top][All Lists]
Advanced

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

Re: [PATCH] Implement join --check-order and --nocheck-order.


From: Pádraig Brady
Subject: Re: [PATCH] Implement join --check-order and --nocheck-order.
Date: Tue, 19 Feb 2008 15:11:21 +0000
User-agent: Thunderbird 2.0.0.6 (X11/20071008)

James Youngman wrote:
> This is a consolidated patch including all the previous changes,
> implementing order checking in join by default.  "make distcheck"
> works (if I move distcheck-hook from Makefile.am to GNUmakefile).

While I like the idea, I'm a little worried about the implementation.
You seem to duplicate the buffers rather than the pointers.
Do you really need to do the extra malloc() + memcpy() per line read?

Could you compare the performance of before and after (with LANG=C).

thanks,
Pádraig.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]