Re: [PATCH v10] ls-files: add eol diagnostics

On 01/11/2016 06:48 PM, Junio C Hamano wrote:

Thanks for comments

quoted

+		if (line_terminator == '\n')
+			printf("i/%-6s w/%-6s attr/%-9s ", i_txt, w_txt, a_txt);

Can we do something better than these hard-coded constants?  Why
can't the "one HT between each" approach be used for both?

v11 will make more clear, that currently "eol=crlf" is the same as "text 
eol=crlf".
The shortest attr is then "", the longest "text eol=crlf".
Using '\t' as a separator makes the output cluttered for a human reader.

When parsing the output, the '\t' is much more convenient (and that's 
what t0027 wil use).
Is there any chance to have a selection between human-friendly and 
script-friendly ?
Would an option -e (eol info using spaces) vs --eol (eol info with TABs) 
make sence ?