invertedtomato / crc

A flexible CRC implementation supporting most major algorithms. Also includes ability to support custom implementations.
MIT License
29 stars 10 forks source link

Add 8-bit bound algorithms covered by e.g. https://docs.rs/crc_all/latest/crc_all/ or https://reveng.sourceforge.io/crc-catalogue not yet covered by `InvertedTomato.Crc` #13

Open maettu-this opened 1 year ago

maettu-this commented 1 year ago

8-bit bound algorithms covered by Rust crc_all not yet covered by InvertedTomato.Crc:

https://reveng.sourceforge.io/crc-catalogue/1-15.htm lists them as well.

https://reveng.sourceforge.io/crc-catalogue/16.htm lists them as well. Interestingly, this link also reveals "CRC-A" as "CRC-16/ISO-IEC-14443-3-A".

https://reveng.sourceforge.io/crc-catalogue/17plus.htm lists them as well. Even more:

Maybe better to extract this into a separate issue.

Originally posted by @maettu-this in https://github.com/invertedtomato/crc/issues/9#issuecomment-1638359255

invertedtomato commented 7 months ago

@maettu-this , sorry for the slow reply on this one. If you made a PR on this I'd be happy to accept it.

maettu-this commented 7 months ago

@invertedtomato, isn't it just adding the e.g....

width=8 poly=0x2f init=0xff refin=false refout=false xorout=0xff check=0xdf residue=0x42 name="CRC-8/AUTOSAR"

...of the mentioned links in the InvertedTomato.Crc required format? Doesn't sound like that much of effort. But when it comes to naming the CRCs in the InvertedTomato.Crc preferred name I think you are the expert much more than me.