wiredtiger/wiredtiger.github.com — Issue #37: cksum review
Author: wiredtiger
Created: 2012-01-31 15:00:32+00:00
State: open

I swapped out the old checksum code I was using in favor of a faster one (the web review indicates it's significantly faster than the one I was using before (Richard Black, algorithm 3), at the cost of 8KB of memory, and a simple measurement of my own confirms this).

However, I did translate the code from Windows to POSIX, and that needs review.

I think there are two things to look at: (1) did I generate the g_crc_slicing array correctly, and (2) did I translate the alignment boundary conditions correctly.

I don't think there's any rush on this, it just needs to happen before we release.