[PATCH] color: allow multiple args to ui.write()

Matt Mackall mpm at selenic.com
Wed Sep 30 18:54:51 CDT 2009


On Tue, 2009-09-29 at 23:59 -0500, Kevin Bullock wrote:
> So here's a patch to make colorwrap() accept multiple arguments. As  
> implemented, it only colorizes the first argument; I'm not sure that's  
> the right way to do it and would welcome feedback.

Yeah, it's not. For correctness, all args should be colorized.
Alternately, we can change ui.write to accept only one arg? Not sure how
often that's used, but it seems like not much.

-- 
http://selenic.com : development and support for Mercurial and Linux




More information about the Mercurial-devel mailing list