Subject: Re: typo?

From: [email-masked]
Date: Thu, 22 Apr 1999 16:45:25 -0700

Noah - I think that is OK.  CRC24_POLY is used in the sample code as:

                   if (crc & 0x1000000)
                       crc ^= CRC24_POLY;

So the extra "1" in CRC24_POLY is actually there to clear the "1" in the
crc variable.  It might have been clearer to define CRC24_POLY without the
"1" and then to do

                   if (crc & 0x1000000)
                       crc ^= CRC24_POLY | 0x1000000;

or even

                   if (crc & 0x1000000) {
                       crc &= 0xffffff;
                       crc ^= CRC24_POLY;
                   }

but it works out the same.

Hal