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

Why a _full_ diff? (was: [PATCH] Char: serial167, cleanup (fwd))

From
Geert Uytterhoeven <geert@linux-m68k.org>
Date
Feb 12, 2007, 08:39 UTC
Message-ID
<Pine.LNX.4.62.0702120934120.10436@chinchilla.sonytel.be>
	Hi,

I received the email below, showing a _full_ (i.e. every single line) diff between the old and the new version:

| Date: Sun, 11 Feb 2007 20:00:54 GMT
| From: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
| To: git-commits-head@vger.kernel.org
| Subject: [PATCH] Char: serial167, cleanup
| 
| Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=44bafdf37b30234671d4e2fb595dea4c3717d089
| Commit:     44bafdf37b30234671d4e2fb595dea4c3717d089
| Parent:     30a063a900518926966f4d75333c1bfbde1658fa
| Author:     Jiri Slaby <jirislaby@gmail.com>
| AuthorDate: Sat Feb 10 01:45:08 2007 -0800
| Committer:  Linus Torvalds <torvalds@woody.linux-foundation.org>
| CommitDate: Sun Feb 11 10:51:28 2007 -0800
| 
|     [PATCH] Char: serial167, cleanup
|     
|     serial167, cleanup
|     
|     - Lindent the code
|     - remove 3 pointers from paranoia_check
|     
|     Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
|     Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
|     Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| ---
|  drivers/char/serial167.c | 5616 +++++++++++++++++++++++-----------------------
|  1 files changed, 2792 insertions(+), 2824 deletions(-)
| 
| diff --git a/drivers/char/serial167.c b/drivers/char/serial167.c
| dissimilarity index 67%
| index af50d32..5fd314a 100644
| --- a/drivers/char/serial167.c
| +++ b/drivers/char/serial167.c
| @@ -1,2824 +1,2792 @@
| -/*
| - * linux/drivers/char/serial167.c
| - *
    [...]
| -#endif
| -
| -MODULE_LICENSE("GPL");
| +/*
| + * linux/drivers/char/serial167.c
| + *
    [...]
| +#endif
| +
| +MODULE_LICENSE("GPL");
| -
| To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
| the body of a message to majordomo@vger.kernel.org
| More majordomo info at  http://vger.kernel.org/majordomo-info.html

At first I thought the new version had CR/LF line endings or so, but that isn't the case.

Just using
| anakin$ git show 44bafdf37b30234671d4e2fb595dea4c3717d089 | diffstat
|  serial167.c | 3322 +++++++++++++++++++++++++++++-------------------------------
|  1 file changed, 1645 insertions(+), 1677 deletions(-)
| anakin$
shows that far from all lines were changed.
Gr{oetje,eeting}s,
						Geert

P.S. I'm not subscribed to git@vger.kernel.org, please CC me on replies -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds
Next: Jeff King
Message 1 of 5 in “Why a _full_ diff? (was: [PATCH] Char: serial167, cleanup (fwd))”
  1. Geert UytterhoevenFeb 12, 2007
  2. Jeff KingFeb 12, 2007
  3. Jeff KingFeb 12, 2007
  4. Jeff KingFeb 12, 2007
  5. Johannes SchindelinFeb 12, 2007

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.