On 7/21/07, Sam Ravnborg <[email protected]> wrote:
On Sat, Jul 21, 2007 at 06:03:00PM -0400, Mike Frysinger wrote: > On 7/21/07, Sam Ravnborg <[email protected]> wrote: > >I would much more prefer this functionality to be integrated into unifdef. > >There is no good reason to have two different preprocesisng methonds, one > >being the sed based one and the other the unidef one. > > > >A sinlge dedicated program that contian the sum of the functionality would > >be faster too. > > which functionality ? normalizing of whitespace or all these > linux-specific hacks ? unifdef serves one specific function which is > stated in its manpage: remove preprocessor conditionals from code. At present the kernel has a private copy of unidef. So adjusting the private copy for the needs of the kernels seems like a god plan.
didnt realize the kernel had integrated it since it didnt at first ... so yeah, extending it however we see fit should be fine -mike - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: "Robert P. J. Day" <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- References:
- [PATCH 22/33] kbuild: do section mismatch check on full vmlinux
- From: Sam Ravnborg <[email protected]>
- [PATCH 24/33] Whitelist references from __dbe_table to .init
- From: Sam Ravnborg <[email protected]>
- [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: Sam Ravnborg <[email protected]>
- Re: [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: Oleg Verych <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: "Mike Frysinger" <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: Oleg Verych <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: "Mike Frysinger" <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: Sam Ravnborg <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: "Mike Frysinger" <[email protected]>
- Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- From: Sam Ravnborg <[email protected]>
- [PATCH 22/33] kbuild: do section mismatch check on full vmlinux
- Prev by Date: Re: More effective processing (Re: [PATCH 25/33] kbuild: use POSIX BRE in headers install target)
- Next by Date: Re: [RFC, Announce] Unified x86 architecture, arch/x86
- Previous by thread: Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- Next by thread: Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target
- Index(es):