Re: [PATCH] diff: add --ignore-blank-lines option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 27, 2013 at 9:14 AM, Antoine Pelisse <apelisse@xxxxxxxxx> wrote:
> On Sun, May 26, 2013 at 10:35 PM, Johannes Sixt <j6t@xxxxxxxx> wrote:
>> I'm thinking of diffs of files with CRLF
>
> Good you did, because I didn't ;-)
>
>> line breaks, where the CR would count as whitespace in the line, I think.
>
> With the current implementation, an empty line with CRLF will not show
> as a blank line if no space option is given. As CR is a space
> according to isspace(3), the line will be removed with any space
> option.

Maybe it would be worth adding the diff(1) "--strip-trailing-cr"
option. So that you could remove empty lines if you have dos line
endings while caring about eol space-changes (or other space changes).
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]