From mboxrd@z Thu Jan  1 00:00:00 1970
To: 9fans@cse.psu.edu
From: Ralph Corderoy <ralph@inputplus.demon.co.uk>
Message-ID: <9l87cb$484$1@inputplus.demon.co.uk>
References: <5.1.0.14.0.20010812233754.00a60bf0@pop.wanadoo.fr>, <Pine.SGI.4.10.10108130043370.758-100000@irixbum.ca>
Subject: Re: [9fans] (no subject)
Date: Tue, 14 Aug 2001 09:44:55 +0000
Topicbox-Message-UUID: dac9fd54-eac9-11e9-9e20-41e7f4b1d025

Hi Andrey,

>         p = pcimatch(nil, MATROX, MGA4xx);
>         if (0 && p)
>         ^^^^^^^^^^^
>                 print("MGA 4xx found : rev %d\n", p->rid);
> 
> i don't quite understand this part..

It looks like a `commented out' print statement used for debug.

The condition was originally `if (p)' so p was only derefenced for
p->rid if it wasn't 0.  Adding the `0 &&' prefix means the condition is
always false, 0 && n is always 0, so the print statement is never
executed.

If the author wants the debug again he can either change the 0 to a 1
or remove the `0 &&'.


Ralph.