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