Adding support for crc-ccitt #4

Merged
merged 1 commit into from Nov 6, 2012

Conversation

Projects
None yet
3 participants
Contributor

underbluewaters commented May 27, 2012

CCITT is very similar to the existing crc16 functions, except that the
initial value is set to 0xFFFF. Test values were verified using pycrc,
and other test strings were verified from developer documentation for
the ReefNet Sensus device I'm working with (though not included in
test/crc.js).

@underbluewaters underbluewaters Adding support for crc-ccitt
CCITT is very similar to the existing crc16 functions, except that the
initial value is set to 0xFFFF. Test values were verified using pycrc,
and other test strings were verified from developer documentation for
the ReefNet Sensus device I'm working with (though not included in
test/crc.js).
7767791
Contributor

underbluewaters commented Jun 14, 2012

Anything I can do to encourage the integration of these changes?

@alexgorbatchev alexgorbatchev added a commit that referenced this pull request Nov 6, 2012

@alexgorbatchev alexgorbatchev Merge pull request #4 from underbluewaters/crc-ccitt-support
Adding support for crc-ccitt
4205239

@alexgorbatchev alexgorbatchev merged commit 4205239 into alexgorbatchev:master Nov 6, 2012

Owner

alexgorbatchev commented Nov 6, 2012

Thank you for contributing!

Contributor

underbluewaters commented Nov 6, 2012

Sweet, first successful pull request :)

Owner

alexgorbatchev commented Nov 6, 2012

haha! ever-ever? congratz! glad i was your first one ;)

jksdua commented Dec 18, 2013

Any reason why this doesn't exist in the latest version?

@alexgorbatchev alexgorbatchev added a commit that referenced this pull request May 5, 2015

@alexgorbatchev alexgorbatchev Merge pull request #4 from underbluewaters/crc-ccitt-support
Adding support for crc-ccitt
eda8419
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment