From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id EB9CF1F61C for ; Mon, 24 Feb 2020 07:33:28 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 2/3] viewdiff: remove optional CR handling Date: Mon, 24 Feb 2020 07:33:27 +0000 Message-Id: <20200224073328.16230-3-e@yhbt.net> In-Reply-To: <20200224073328.16230-1-e@yhbt.net> References: <20200224073328.16230-1-e@yhbt.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: The only caller of `flush_diff' is `add_text_body', and that already did CRLF conversion on the text part. The regexps in SolverGit still need to preserve CR, however, since that actually applies patches (instead of rendering them), and we need to preserve CRLF patches for CRLF files. --- lib/PublicInbox/ViewDiff.pm | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/PublicInbox/ViewDiff.pm b/lib/PublicInbox/ViewDiff.pm index 604b1f23..0f5c0e4e 100644 --- a/lib/PublicInbox/ViewDiff.pm +++ b/lib/PublicInbox/ViewDiff.pm @@ -150,8 +150,8 @@ sub diff_header ($$$$) { sub diff_before_or_after ($$$) { my ($dst, $ctx, $x) = @_; my $linkify = $ctx->{-linkify}; - for my $y (split(/(^---\r?\n)/sm, $$x)) { - if ($y =~ /\A---\r?\n\z/s) { + for my $y (split(/(^---\n)/sm, $$x)) { + if ($y =~ /\A---\n\z/s) { $$dst .= "---\n"; # all HTML is "\r\n" => "\n" } elsif ($y =~ /^ [0-9]+ files? changed, /sm) { # ok, looks like a diffstat, go line-by-line: @@ -167,11 +167,12 @@ sub diff_before_or_after ($$$) { } } +# callers must do CRLF => LF conversion before calling this sub flush_diff ($$$) { my ($dst, $ctx, $cur) = @_; - state $LF = qr!\r?\n!; - state $ANY = qr![^\r\n]!; - state $FN = qr!(?:"?[^/\n]+/[^\r\n]+|/dev/null)!; + state $LF = qr!\n!; + state $ANY = qr![^\n]!; + state $FN = qr!(?:"?[^/\n]+/[^\n]+|/dev/null)!; my @top = split(/( (?: # begin header stuff, don't capture filenames, here,