git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 03/10] diff.c: drop tautologous condition in emit_line_0

From
Stefan Beller <stefanbeller@gmail.com>
Date
Sep 11, 2016, 05:42 UTC
Message-ID
<1473572530-25764-4-git-send-email-stefanbeller@gmail.com>
In-Reply-To
<1473572530-25764-1-git-send-email-stefanbeller@gmail.com>
From: Stefan Beller <sbeller@google.com>

When `first` equals '\r', then it cannot equal '\n', which makes `!has_trailing_newline` always true if `first` is '\r'. Drop that check.

Signed-off-by: Stefan Beller <sbeller@google.com>
---
 diff.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/diff.c b/diff.c
index 156c2aa..9d2e704 100644
--- a/diff.c
+++ b/diff.c
@@ -460,8 +460,7 @@ static void emit_line_0(struct diff_options *o, const char *set, const char *res
 
 	if (len == 0) {
 		has_trailing_newline = (first == '\n');
-		has_trailing_carriage_return = (!has_trailing_newline &&
-						(first == '\r'));
+		has_trailing_carriage_return = (first == '\r');
 		nofirst = has_trailing_newline || has_trailing_carriage_return;
 	} else {
 		has_trailing_newline = (len > 0 && line[len-1] == '\n');
-- 
2.7.4
Previous: Junio C HamanoNext: Junio C Hamano
Message 6 of 21 in “Another preparatory series for rename detection”
  1. 00/10 Another preparatory series for rename detectionStefan Beller, Sep 11, 2016
  2. 01/10 diff: move line ending check into emit_hunk_headerStefan Beller, Sep 11, 2016
  3. Junio C HamanoSep 12, 2016
  4. 02/10 diff: emit_{add, del, context}_line to increase {pre,post}image line countStefan Beller, Sep 11, 2016
  5. Junio C HamanoSep 12, 2016
  6. 03/10 diff.c: drop tautologous condition in emit_line_0Stefan Beller, Sep 11, 2016
  7. Junio C HamanoSep 12, 2016
  8. Stefan BellerSep 16, 2016
  9. 05/10 diff.c: reintroduce diff_flush_patch for all filesStefan Beller, Sep 11, 2016
  10. Junio C HamanoSep 13, 2016
  11. 04/10 diff.c: rename diff_flush_patch to diff_flush_patch_filepairStefan Beller, Sep 11, 2016
  12. Junio C HamanoSep 12, 2016
  13. 07/10 diff.c: convert fn_out_consume to use emit_line_*Stefan Beller, Sep 11, 2016
  14. Junio C HamanoSep 13, 2016
  15. Stefan BellerSep 13, 2016
  16. 10/10 submodule.c: convert show_submodule_summary to use emit_line_fmtStefan Beller, Sep 11, 2016
  17. 09/10 diff.c: convert emit_rewrite_lines to use emit_line_0Stefan Beller, Sep 11, 2016
  18. 08/10 diff.c: convert emit_rewrite_diff to use emit_line_*Stefan Beller, Sep 11, 2016
  19. 06/10 diff.c: emit_line_0 can handle no colorStefan Beller, Sep 11, 2016
  20. Junio C HamanoSep 13, 2016
  21. Stefan BellerSep 13, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.