Re: [PATCH 2/4] cat-file: introduce the --filters option

From: Torsten Bögershausen <tboegi@web.de>
Date: Aug 22, 2016, 16:51 UTC
Message-ID: <0ee82126-3fae-723a-ab89-b5794c4abc74@web.de>
In-Reply-To: <alpine.DEB.2.20.1608191656080.4924@virtualbox>

On 19.08.16 17:00, Johannes Schindelin wrote:

Show 20 quoted lines

> Hi Torsten,
>
> On Fri, 19 Aug 2016, Torsten Bögershausen wrote:
>
>> On Thu, Aug 18, 2016 at 02:46:17PM +0200, Johannes Schindelin wrote:
>>
>>> +--filters::
>>> +	Show the content as transformed by the filters configured in
>> Minor comment:
>> s/transformed/converted/ ?
> Sure.
>
>> Does it make sense to be more specific here:
>> The order of conversion is
>> - ident
>> - CRLF
>> - smudge
> I do not think it makes sense to complexify the documentation in that
> manner. The filters should always be applied in the same order, methinks,
> and it would only clutter the man page to repeat that order here.

Can we can shorten the description and have something like this:

--filters::
+	Show the content converted by the filters configured in
+	the current working tree for the given <path>. 
+ 	<object> has to be of the form <tree-ish>:<path> or :<path>.
+