[PATCH] add color diff support to record extension

Dirkjan Ochtman dirkjan at ochtman.nl
Mon Aug 10 04:49:27 CDT 2009


On Mon, Aug 10, 2009 at 11:45, Dan Villiom Podlaski
Christiansen<danchr at gmail.com> wrote:
> Wouldn't it be worthwhile to integrate colour support into core? Currently,
> the extension either duplicates the functionality of the commands it wraps,
> or parses their output. It seems likely that having the commands themselves
> add styles would make the implementation both simpler and more accurate.

Would be fine with me as I already enable it everywhere anyway. I'll
let Matt decide, though.

> With regard to ‘record’, wouldn't it be an excellent candidate for
> integration as well? After all, it's hardly complex nor dangerous…

Even if it's not complex, it does add an extra command, which makes
the UI more complex.

Cheers,

Dirkjan



More information about the Mercurial-devel mailing list