* [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®