Change counter to uint64_t

[real-or-random]

I hope this fixes #2. I haven't tested this.

[markpeek]

@samrushing can you review, test, and approve?

[samrushing]

Sorry for the delay! The bitcount is obviously being packed as a 64-bit quantity. Although I found some C implementations using 'int' or 'uint32_t', uint64_t is obviously correct. That's also what's used in NaCl/libsodium.

[samrushing]

Heh, I added an 8GB test and it took 2200s to fail. You sure you want a test? 8^)

[markpeek]

Technically I did get you to test it. ;-) Are you going to hit "Approve" through "Review Changes"?

[samrushing]

diff --git a/tests/test_sha256.py b/tests/test_sha256.py
index 6290ea3..968f993 100644
--- a/tests/test_sha256.py
+++ b/tests/test_sha256.py
@@ -50,6 +50,14 @@ class TestSHA256(unittest.TestCase):
                     s = new
             self.assertEquals(s.hexdigest(), hash)
 
+    def test_counter(self):
+        s = 'X' * 1048576
+        h0 = sha()
+        h1 = hsha()
+        for i in range (512):
+            h0.update (s)
+            h1.update (s)
+        self.assertEquals (h0.hexdigest(), h1.hexdigest())
 
 if __name__ == '__main__':
     unittest.main()

That takes about 150s to run, add it if you dare!

[markpeek]

Thanks!