From 97bbdd21d38d9063e99be53c489f084bbbf23d76 Mon Sep 17 00:00:00 2001
From: forrest <forrest@470744a7-cac9-478e-843e-5ec1b25c69e8>
Date: Tue, 12 Jul 2011 07:46:33 +0000
Subject: [PATCH] binary cleanup

git-svn-id: svn://forre.st/p2pool@1378 470744a7-cac9-478e-843e-5ec1b25c69e8
---
 p2pool/bitcoin/data.py |   17 +++--------------
 1 files changed, 3 insertions(+), 14 deletions(-)

diff --git a/p2pool/bitcoin/data.py b/p2pool/bitcoin/data.py
index eb94dfd..930d27a 100644
--- a/p2pool/bitcoin/data.py
+++ b/p2pool/bitcoin/data.py
@@ -121,8 +121,7 @@ class VarStrType(Type):
         return read(file, length)
     
     def write(self, file, item):
-        file = self._inner_size.write(file, len(item))
-        return file, item
+        return self._inner_size.write(file, len(item)), item
 
 class FixedStrType(Type):
     def __init__(self, length):
@@ -133,7 +132,7 @@ class FixedStrType(Type):
     
     def write(self, file, item):
         if len(item) != self.length:
-            raise ValueError('incorrect length!')
+            raise ValueError('incorrect length item!')
         return file, item
 
 class EnumType(Type):
@@ -172,7 +171,7 @@ class ShortHashType(Type):
         return int(data[::-1].encode('hex'), 16), file
     
     def write(self, file, item):
-        if item >= 2**160:
+        if not 0 <= item < 2**160:
             raise ValueError("invalid hash value")
         return file, ('%040x' % (item,)).decode('hex')[::-1]
 
@@ -196,15 +195,6 @@ class ListType(Type):
             file = self.type.write(file, subitem)
         return file
 
-class FastLittleEndianUnsignedInteger(Type):
-    def read(self, file):
-        data, file = read(file, 4)
-        data = map(ord, data)
-        return data[0] + (data[1] << 8) + (data[2] << 16) + (data[3] << 24), file
-    
-    def write(self, file, item):
-        return StructType("<I").write(file, item)
-
 class StructType(Type):
     def __init__(self, desc):
         self.desc = desc
@@ -274,7 +264,6 @@ class FloatingIntegerType(Type):
     # redundancy doesn't matter here because bitcoin checks binary bits against its own computed bits
     # so it will always be encoded 'normally' in blocks (they way bitcoin does it)
     _inner = StructType("<I")
-    _inner = FastLittleEndianUnsignedInteger()
     
     def read(self, file):
         bits, file = self._inner.read(file)
-- 
1.7.1