libnxz / power-gzip

POWER NX zlib compliant library
23 stars 18 forks source link

Add support for big endian crc32 #196

Closed ghost closed 2 years ago

ghost commented 2 years ago

This is copied from [1] with minor changes since no support is needed for reflected crcs, just big-endian and little-endian.

This is work towards fixing #191

[1] https://github.com/antonblanchard/crc32-vpmsum/

Signed-off-by: Alan Mock alan@alanmock.com

ghost commented 2 years ago

@tuliom thanks for the review. I think I have made all the requested changes.

ghost commented 2 years ago

@tuliom can you take another look at this? I'd like to get it merged before I move on to the next part.

RajalakshmiSR commented 2 years ago

LGTM. @RajalakshmiSR , any other comments?

I merged it, Thanks.