On Tue, 2008-02-12 at 17:38 -0500, Kenneth Lerman wrote:
> The function CRC16 is used before it is declared.
> 
> Either move the definition of CRC16 to before main, or declare it separately 
> prior to its use.

I moved main to the bottom of the file, and it compiled without
complaint. Thank you.

> I notice that you are using (very) old style declarations in that code. 
> Modern coders would write:
> unsigned short CRC16(unsigned char *puchMsg, unsigned short usDataLen)etcKen

I stole my CRC16 from the Modbus-IDA.org literature. I notice that the
Classic Ladder CRC16 uses the modern style. More stuff to study, but I
kind of like the old ways. It took me while to change from linksp to
net.

-- 
Kirk Wallace (California, USA
http://www.wallacecompany.com/machine_shop/ 
Hardinge HNC lathe,
Bridgeport mill conversion, doing XY now,
Zubal lathe conversion pending)


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Emc-users mailing list
Emc-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/emc-users

Reply via email to