Re: [avr-libc-dev] [patch #7485] CRC8-CCITT

[Dave Hylands]

Hi Frédéric,

2011/2/27 Frédéric Nadeau <address@hidden>:
> URL:
>  <http://savannah.nongnu.org/patch/?7485>
>
>                 Summary: CRC8-CCITT
>                 Project: AVR C Runtime Library
>            Submitted by: fnadeau
>            Submitted on: lun. 28 févr. 2011 02:50:49 GMT
>                Category: None
>                Priority: 5 - Normal
>                  Status: None
>                 Privacy: Public
>             Assigned to: None
>        Originator Email:
>             Open/Closed: Open
>         Discussion Lock: Any
>
>    _______________________________________________________
>
> Details:
>
> This patch add _crc8_ccitt_update(). The name might be changed, or
> crc_ccitt_update() could be updated to crc16_ccitt_update to avoid possible
> confusion.

Just one really minor quibble.

The C code is missing a line.

It should have

return data;

Dave Hylands