Apologies (Re: This doesn't belong in 2.0.x !!!)

Arthur D. Jerijian (celestra@ix.netcom.com)
Tue, 09 Jul 1996 02:17:22 GMT

I realize that I responded a bit prematurely after downloading the new
kernel. I knew I should have let myself calm down before bashing
Linus and Hannu (you guys are doing a GREAT job, BTW, so don't feel
too guilty about the semicolon bug). It's just that I don't like to
see stuff like this happen in 2.0.x, that's all. I realize that we're
all human and we make mistakes from time to time. Hell, I make
mistakes *ALL* the time (by flaming others prematurely, that is).
Apologies for the mess.

--Arthur

On Mon, 8 Jul 1996 11:39:08 -0600 (MDT), you wrote:

>Phil Lewis dons pointy hat and quotes thusly:
>
>Don't have a cow, man.

>> Linus, Hannu, I appreciate all the effors that you're trying
>>to put into making Linux a high-quality kernel. But, why the h*ll
>>are you trying to change the sound driver so much? It was fine the
>>way it was in 2.0.0 before the update! In 2.0.4, I get the following
>>COMPILE-TIME ERROR:

>Whole buncha compiler messages deleted.
>
>> This is all because of ONE SEMICOLON, which is fixed in the
>>following patch. I thought that errors like this aren't supposed to
>>happen in production kernels such as 2.0.x.

>Gee Whiz! If you had paid $350.00US for it you'd have room to complain.
>I, personally, am glad to have Linux missing semicolons and all.
>I mean, well, if it were not for Linux I might be roaming the streets
>looking for drugs or prostitutes or something. Hannu and Linus have my
>highest admiration (apple-polishing mode off).
>
>Arthur's patch is included below for those who have not used it yet.
>
>Phil Lewis (Lord of the Lamers)
>
>

>--- linux/drivers/sound/sb_common.c.orig Mon Jul 8 06:56:14 1996
>+++ linux/drivers/sound/sb_common.c Mon Jul 8 06:56:22 1996
>@@ -1173,7 +1173,7 @@
> }
> hw_config->name = "Sound Blaster 16";
> hw_config->irq = -devc->irq;
>- sb16_set_mpu_port(devc, hw_config)
>+ sb16_set_mpu_port(devc, hw_config);
> break;
>
> case MDL_ESS:

> --Arthur