* [PATCH] 2.5.43 : drivers/block/xd.c
@ 2002-10-19 0:58 Frank Davis
2002-10-20 23:33 ` Daniel Egger
0 siblings, 1 reply; 2+ messages in thread
From: Frank Davis @ 2002-10-19 0:58 UTC (permalink / raw)
To: linux-kernel; +Cc: fdavis, torvalds
Hello all,
The following fixes a 'used but not declared' compile error. Please review
for inclusion.
Regards,
Frank
--- linux/drivers/block/xd.c.old Fri Oct 18 20:51:44 2002
+++ linux/drivers/block/xd.c Fri Oct 18 20:54:02 2002
@@ -150,6 +150,7 @@
static int __init xd_init(void)
{
u_char i,controller;
+ u_char count = 0;
unsigned int address;
int err;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] 2.5.43 : drivers/block/xd.c
2002-10-19 0:58 [PATCH] 2.5.43 : drivers/block/xd.c Frank Davis
@ 2002-10-20 23:33 ` Daniel Egger
0 siblings, 0 replies; 2+ messages in thread
From: Daniel Egger @ 2002-10-20 23:33 UTC (permalink / raw)
To: Frank Davis; +Cc: linux-kernel, torvalds
[-- Attachment #1: Type: text/plain, Size: 581 bytes --]
Am Sam, 2002-10-19 um 02.58 schrieb Frank Davis:
> Hello all,
> The following fixes a 'used but not declared' compile error. Please review
> for inclusion.
The "standard" way of achieving the same would be to assign the variable
to itself. In your fragment the compiler will initialise the variable to
zero which will cost at least one instruction and thus "bloat" the code,
it'll also be slower due to an additional instruction and may schedule
worse. Not that it mattered much in this case but if you can save a byte
or two... :)
--
Servus,
Daniel
[-- Attachment #2: Dies ist ein digital signierter Nachrichtenteil --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2002-10-21 7:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-10-19 0:58 [PATCH] 2.5.43 : drivers/block/xd.c Frank Davis
2002-10-20 23:33 ` Daniel Egger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®