Messages in this thread |  | | From | Keith Owens <> | Subject | Re: [PATCH] Fix compile warnings in 2.4.0 | Date | Mon, 08 Jan 2001 11:52:27 +1100 |
| |
On Sun, 7 Jan 2001 16:19:50 -0500, "Rich Baum" <baumr1@coral.indstate.edu> wrote: >This patch should fix the rest of the warnings about #endif >statements when using the 20001225 gcc snapshot. Thanks to >Keith Owens for providing a script to automate this process. It got >the job done sooner and found warnings to fix for non x86 platforms.
As reported by Neil Booth, your patch contains errors which do not appear in my patched system using my script. You probably skipped the leading '^' in the regexp. It also turns out that there are a couple of m68k assembler lines which have a trailing '#' which starts a comment on that assembler. So treat '#' as a comment marker as well.
Whatever you used, it was not my script. It should be (with '#' added)
find -type f -name '*.[chS]' | \ xargs perl -lpi -e 's:^(\s*#\s*endif)\s+([^/\s#].*)$:\1\t/* \2 */:;'
BTW, until the patch is correct do not bother sending it to Alan Cox or Linus, just to linux-kernel..
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
|  |