Match CRC from C code to crc-full

[riologiuseppe]

Match CRC from C code to crc-full

[RioloGiuseppe]

You're welcome!
Yes. To do it fast I used this online tool, so OPN became 0x4F, 0x50, 0x4E and \x17 is an escaped form of 0x17 (I don 't know in which convention).

If I may, I'm whiling to contribute an update to your repo and package so it can be easily consumed from npm. Right now, you are asking people to install typescript, which shouldn't be required since they may want to use it on JS as well.

Yes, you are right, including the compiled javascript tsc will not a dependency any more. You can make a PR when you want.

Best regards!

[RioloGiuseppe]

Hello,
I just made some tries. I think you get a wrong result because you are passing "OPN\x17" as string. If you pass the equivalent byte array of ascii characters, you will havethe same result of C/C++.

As in the readme:

import {CRC} from 'crc-full'
var crc = new CRC(16, "CRC16", 0x1021, 0x0000, 0x0000, false, false);
var res = crc .compute([0x4F, 0x50, 0x4E, 0x17]);        // Array with ascii integer of "OPN\x17"

The result (number) is 43177 but if you convert to an hex string you will have: 'a8a9', that is the same result obtained in C.

h_string = res.toString(16)

Hope to been useful
Best regards
Giuseppe Riolo

[galvesribeiro]

@RioloGiuseppe hello! Thank you very much for the quick response.

I got it working!

If I may, I'm whiling to contribute an update to your repo and package so it can be easily consumed from npm. Right now, you are asking people to install typescript, which shouldn't be required since they may want to use it on JS as well.

If you accept the PR and will update the npm package, I can push it later Today.

Thanks!