On 03/02/2012 01:47 PM, Sage Weil wrote:
These look good.. but where is the crc bug actually fixed? Not seeing
that patch...
I haven't addressed it yet. Soon, very soon...
-Alex
Reviewed-by: Sage Weil<sage@xxxxxxxxxxxx>
On Tue, 28 Feb 2012, Alex Elder wrote:
Rename a few variables so it's clearer which indicate that a
CRC should be computed and which hold an actual CRC value.
Separate CRC calculation from byte swapping to improve
readability. And move some code that runs on only on
the last pass through a couple of loops outside the loop
body.
-Alex
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
