Re: [PATCH] Trivial, add missing newline at EOF inDocumentation/networking/packet_mmap.txt

From: Jesper Juhl
Date: Tue Jun 08 2004 - 18:12:27 EST


On Sun, 6 Jun 2004, Pavel Machek wrote:

> Hi!
>
> > Missing newlines at the end of files make them less pleasing to work with
> > for a number of tools that work on a line-by-line basis, and for source files
> > it will cause gcc to emit a warning. So, I desided to add that missing
> > newline to the few files in the kernel that are missing it.
> > This patch makes no functional changes at all to the kernel.
> > Patch is against 2.6.7-rc2
> >
> > Here's the patch adding a newline to
> > Documentation/networking/packet_mmap.txt
>
> Perhaps you should strip headers/lkml signature of packet_mmap.txt?
> Pavel

That makes sense to me, here's an updated patch against 2.6.7-rc3
(adding David S. Miller and Ulisses Alonso Camaró to the CC list since the
comment I'm removing at the top involves them (hope I got the right
persons behind "DaveM" & "Ulisses" there)) :


--- linux-2.6.7-rc3/Documentation/networking/packet_mmap.txt-orig 2004-06-09 00:58:12.000000000 +0200
+++ linux-2.6.7-rc3/Documentation/networking/packet_mmap.txt 2004-06-09 00:58:30.000000000 +0200
@@ -1,11 +1,4 @@

-DaveM:
-
-If you agree with it I will send two small patches to modify
-kernel's configure help.
-
- Ulisses
-
--------------------------------------------------------------------------------
+ ABSTRACT
--------------------------------------------------------------------------------
@@ -405,8 +398,3 @@ then poll for frames.

Jesse Brandeburg, for fixing my grammathical/spelling errors

->>> EOF
--
-To unsubscribe from this list: send the line "unsubscribe linux-net" in
-the body of a message to majordomo@xxxxxxxxxxxxxxx
-More majordomo info at http://vger.kernel.org/majordomo-info.html
\ No newline at end of file


--
Jesper Juhl <juhl-lkml@xxxxxx>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/