* [PATCH 1/3] compilation fixes for gcc 4.0
@ 2004-09-22 19:19 Grzegorz Piotr Jaskiewicz
2004-09-23 1:58 ` Grzegorz Piotr Jaskiewicz
0 siblings, 1 reply; 2+ messages in thread
From: Grzegorz Piotr Jaskiewicz @ 2004-09-22 19:19 UTC (permalink / raw)
To: Andrew Morton; +Cc: kernel list
gcc 4.0 doesn't treat this extension as valid anymore.
one of the gcc guys said to me:
" For that matter, they could neatly avoid the whole issue by writing ANSI/ISO-compliant code instead of
using GCC extensions."
That should be sufficient for explanation.
In all these cases, I see no reason to declare prototype inside body of function.
It makes code less readable.
I am sure it's present in few more places, but that's the only place where I can spot it.
Please apply.
Signed-off-by: Grzegorz Jaskiewicz <gj at pointblue.com.pl>
--- drivers/input/joystick/grip_mp.c 2004-08-14 07:37:42 +0200
+++ /tmp/grip_mp.c 2004-09-22 20:58:40 +0200
@@ -107,6 +107,8 @@
static int axis_map[] = { 5, 9, 1, 5, 6, 10, 2, 6, 4, 8, 0, 4, 5, 9, 1,
5 };
+static void register_slot(int i, struct grip_mp *grip);
+
/*
* Returns whether an odd or even number of bits are on in pkt.
*/
@@ -355,7 +357,6 @@
u32 packet;
int joytype = 0;
int slot = 0;
- static void register_slot(int i, struct grip_mp *grip);
/* Get a packet and check for validity */
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 1/3] compilation fixes for gcc 4.0
2004-09-22 19:19 [PATCH 1/3] compilation fixes for gcc 4.0 Grzegorz Piotr Jaskiewicz
@ 2004-09-23 1:58 ` Grzegorz Piotr Jaskiewicz
0 siblings, 0 replies; 2+ messages in thread
From: Grzegorz Piotr Jaskiewicz @ 2004-09-23 1:58 UTC (permalink / raw)
To: Andrew Morton; +Cc: kernel list
[-- Attachment #1: Type: text/plain, Size: 691 bytes --]
Grzegorz Piotr Jaskiewicz wrote:
> gcc 4.0 doesn't treat this extension as valid anymore.
>
> one of the gcc guys said to me:
> " For that matter, they could neatly avoid the whole issue by writing
> ANSI/ISO-compliant code instead of
> using GCC extensions."
> That should be sufficient for explanation.
>
> In all these cases, I see no reason to declare prototype inside body
> of function. It makes code less readable.
>
> I am sure it's present in few more places, but that's the only place
> where I can spot it.
> Please apply.
>
This is resend, mozilla seem to do wordwraping. Patch is attached.
Signed-off-by: Grzegorz Jaskiewicz <gj at pointblue.com.pl>
---
--
GJ
[-- Attachment #2: grip_mp.c.patch --]
[-- Type: text/plain, Size: 540 bytes --]
--- a/drivers/input/joystick/grip_mp.c 2004-08-14 07:37:42 +0200
+++ b/drivers/input/joystick/grip_mp.c 2004-09-22 20:58:40 +0200
@@ -107,6 +107,8 @@
static int axis_map[] = { 5, 9, 1, 5, 6, 10, 2, 6, 4, 8, 0, 4, 5, 9, 1, 5 };
+static void register_slot(int i, struct grip_mp *grip);
+
/*
* Returns whether an odd or even number of bits are on in pkt.
*/
@@ -355,7 +357,6 @@
u32 packet;
int joytype = 0;
int slot = 0;
- static void register_slot(int i, struct grip_mp *grip);
/* Get a packet and check for validity */
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-09-23 1:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-09-22 19:19 [PATCH 1/3] compilation fixes for gcc 4.0 Grzegorz Piotr Jaskiewicz
2004-09-23 1:58 ` Grzegorz Piotr Jaskiewicz
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.