Re: [PATCH] column: Fix an incorrect parse of the 'nodense' option token

From

Ramsay Jones <ramsay@ramsay1.demon.co.uk>

Date

Feb 14, 2012, 18:28 UTC

Message-ID

<4F3AA7D2.9040702@ramsay1.demon.co.uk>

In-Reply-To

<CACsJy8B7=6OYBV-HVK3Cq4_cTdPvm_1MvVv=U2R=4yQw9t5X9Q@mail.gmail.com>

Nguyen Thai Ngoc Duy wrote:

> It's about overriding config. If you set "dense" by default in
> column.ui but do not want it in this particular run, you can say
> --column=nodense.

Ah, OK, I missed that (obvious in retrospect!). Thanks.

> The [no]color is for plumbing only. If a command produces colored
> output, "color" is required to calculate text length correctly.
> Overriding it with "nocolor" would break the layout badly so it's no
> use there. It does not make sense (to me) for users to put "color" in
> column.ui. Which is why it's not mentioned in document.