[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Roman Zippel
Hi, On Tue, 21 Jun 2005, Pierre Ossman wrote: A (somewhat unclean) solution is to make the type change based on the platform. Are there any defines present to test if we're in a Solaris environment? I don't have access to any Solaris machines myself so I can't really test. Just ignore it.

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Roman Zippel
Hi, On Tue, 21 Jun 2005, Pierre Ossman wrote: Should I extract the changes for bkbits and make a reversed patch? No, go through the warnings, analyze each one and choose an appropriate solution. You might want to keep notes, which you can post with the changelogs, so one can reproduce, why a

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Roman Zippel
Hi, On Tue, 21 Jun 2005, Pierre Ossman wrote: Is there some easy way to check the file history? Downloading a couple of old kernels just for one file is a bit of a hassle. And I don't run bk so I can't access that repository (is it even still present after Linus' move?).

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Roman Zippel
Hi, On Tue, 21 Jun 2005, Pierre Ossman wrote: GCC 4 checks the signedness of pointer casts and generates a whole bunch of warnings for code in scripts/ (which makes heavy use of signed char strings). This patch adds explicit casts. Just adding explicit all over the place is really the

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Bernd Petrovitsch
On Tue, 2005-06-21 at 14:59 +0200, Pierre Ossman wrote: [...] It should only be a matter of reversing the patches for Solaris then. But that will of course bring back the warnings on that platform. I'd say we should stick with what the standard says. Unfortunatly I don't The C-standard about

[kbuild-devel] [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Pierre Ossman
GCC 4 checks the signedness of pointer casts and generates a whole bunch of warnings for code in scripts/ (which makes heavy use of signed char strings). This patch adds explicit casts. Signed-off-by: Pierre Ossman [EMAIL PROTECTED] Index: linux-wbsd/scripts/basic/fixdep.c

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Andreas Schwab
Pierre Ossman [EMAIL PROTECTED] writes: Is there some easy way to check the file history? Downloading a couple of old kernels just for one file is a bit of a hassle. And I don't run bk so I can't access that repository You can use the web interface (bkbits.net). Andreas. -- Andreas Schwab,

[kbuild-devel] Re: [PATCH] Pointer cast warnings in scripts/

2005-06-21 Thread Mikael Pettersson
Pierre Ossman writes: GCC 4 checks the signedness of pointer casts and generates a whole bunch of warnings for code in scripts/ (which makes heavy use of signed char strings). This patch adds explicit casts. This is way ugly. The _real_ bug is that someone decided to store plain text