10 Jan
2011
10 Jan
'11
7:34 p.m.
On Mon, Jan 10, 2011 at 08:23:23PM +0100, Clément Bœsch wrote:
Doesn't seem like much of an improvement but I don't mind. However for this second patch I'd prefer if you applied merging the two if(!quiet) separately, also because the patch you have attached ends up with the comment at a wrong and confusing place.
Ok, I'll send a new patch. Btw, what is the "handle this better" refering to?
I suspect the duplicated !quiet condition and generally that code was considered a bit ugly by whoever wrote it.