Use python's set type, rather than a dict with all values == True

Jeff Garzik · Jeff Garzik · commit 05a076b58c01 · 2012-08-10T13:05:48.000-04:00
diff --git a/bitcoin/script.py b/bitcoin/script.py
@@ -154,129 +154,130 @@
 OP_INVALIDOPCODE = 0xff
 
 VALID_OPCODES = {
-	OP_1NEGATE : True,
-	OP_RESERVED : True,
-	OP_1 : True,
-	OP_2 : True,
-	OP_3 : True,
-	OP_4 : True,
-	OP_5 : True,
-	OP_6 : True,
-	OP_7 : True,
-	OP_8 : True,
-	OP_9 : True,
-	OP_10 : True,
-	OP_11 : True,
-	OP_12 : True,
-	OP_13 : True,
-	OP_14 : True,
-	OP_15 : True,
-	OP_16 : True,
-
-	OP_NOP : True,
-	OP_VER : True,
-	OP_IF : True,
-	OP_NOTIF : True,
-	OP_VERIF : True,
-	OP_VERNOTIF : True,
-	OP_ELSE : True,
-	OP_ENDIF : True,
-	OP_VERIFY : True,
-	OP_RETURN : True,
-
-	OP_TOALTSTACK : True,
-	OP_FROMALTSTACK : True,
-	OP_2DROP : True,
-	OP_2DUP : True,
-	OP_3DUP : True,
-	OP_2OVER : True,
-	OP_2ROT : True,
-	OP_2SWAP : True,
-	OP_IFDUP : True,
-	OP_DEPTH : True,
-	OP_DROP : True,
-	OP_DUP : True,
-	OP_NIP : True,
-	OP_OVER : True,
-	OP_PICK : True,
-	OP_ROLL : True,
-	OP_ROT : True,
-	OP_SWAP : True,
-	OP_TUCK : True,
-
-	OP_CAT : True,
-	OP_SUBSTR : True,
-	OP_LEFT : True,
-	OP_RIGHT : True,
-	OP_SIZE : True,
-
-	OP_INVERT : True,
-	OP_AND : True,
-	OP_OR : True,
-	OP_XOR : True,
-	OP_EQUAL : True,
-	OP_EQUALVERIFY : True,
-	OP_RESERVED1 : True,
-	OP_RESERVED2 : True,
-
-	OP_1ADD : True,
-	OP_1SUB : True,
-	OP_2MUL : True,
-	OP_2DIV : True,
-	OP_NEGATE : True,
-	OP_ABS : True,
-	OP_NOT : True,
-	OP_0NOTEQUAL : True,
-
-	OP_ADD : True,
-	OP_SUB : True,
-	OP_MUL : True,
-	OP_DIV : True,
-	OP_MOD : True,
-	OP_LSHIFT : True,
-	OP_RSHIFT : True,
-
-	OP_BOOLAND : True,
-	OP_BOOLOR : True,
-	OP_NUMEQUAL : True,
-	OP_NUMEQUALVERIFY : True,
-	OP_NUMNOTEQUAL : True,
-	OP_LESSTHAN : True,
-	OP_GREATERTHAN : True,
-	OP_LESSTHANOREQUAL : True,
-	OP_GREATERTHANOREQUAL : True,
-	OP_MIN : True,
-	OP_MAX : True,
-
-	OP_WITHIN : True,
-
-	OP_RIPEMD160 : True,
-	OP_SHA1 : True,
-	OP_SHA256 : True,
-	OP_HASH160 : True,
-	OP_HASH256 : True,
-	OP_CODESEPARATOR : True,
-	OP_CHECKSIG : True,
-	OP_CHECKSIGVERIFY : True,
-	OP_CHECKMULTISIG : True,
-	OP_CHECKMULTISIGVERIFY : True,
-
-	OP_NOP1 : True,
-	OP_NOP2 : True,
-	OP_NOP3 : True,
-	OP_NOP4 : True,
-	OP_NOP5 : True,
-	OP_NOP6 : True,
-	OP_NOP7 : True,
-	OP_NOP8 : True,
-	OP_NOP9 : True,
-	OP_NOP10 : True,
-
-	OP_SMALLINTEGER : True,
-	OP_PUBKEYS : True,
-	OP_PUBKEYHASH : True,
-	OP_PUBKEY : True,
+	OP_1NEGATE,
+	OP_RESERVED,
+	OP_1,
+	OP_2,
+	OP_3,
+	OP_4,
+	OP_5,
+	OP_6,
+	OP_7,
+	OP_8,
+	OP_9,
+	OP_10,
+	OP_11,
+	OP_12,
+	OP_13,
+	OP_14,
+	OP_15,
+	OP_16,
+
+	OP_NOP,
+	OP_VER,
+	OP_IF,
+	OP_NOTIF,
+	OP_VERIF,
+	OP_VERNOTIF,
+	OP_ELSE,
+	OP_ENDIF,
+	OP_VERIFY,
+	OP_RETURN,
+
+	OP_TOALTSTACK,
+	OP_FROMALTSTACK,
+	OP_2DROP,
+	OP_2DUP,
+	OP_3DUP,
+	OP_2OVER,
+	OP_2ROT,
+	OP_2SWAP,
+	OP_IFDUP,
+	OP_DEPTH,
+	OP_DROP,
+	OP_DUP,
+	OP_NIP,
+	OP_OVER,
+	OP_PICK,
+	OP_ROLL,
+	OP_ROT,
+	OP_SWAP,
+	OP_TUCK,
+
+	OP_CAT,
+	OP_SUBSTR,
+	OP_LEFT,
+	OP_RIGHT,
+	OP_SIZE,
+
+	OP_INVERT,
+	OP_AND,
+	OP_OR,
+	OP_XOR,
+	OP_EQUAL,
+	OP_EQUALVERIFY,
+	OP_RESERVED1,
+	OP_RESERVED2,
+
+	OP_1ADD,
+	OP_1SUB,
+	OP_2MUL,
+	OP_2DIV,
+	OP_NEGATE,
+	OP_ABS,
+	OP_NOT,
+	OP_0NOTEQUAL,
+
+	OP_ADD,
+	OP_SUB,
+	OP_MUL,
+	OP_DIV,
+	OP_MOD,
+	OP_LSHIFT,
+	OP_RSHIFT,
+
+	OP_BOOLAND,
+	OP_BOOLOR,
+	OP_NUMEQUAL,
+	OP_NUMEQUALVERIFY,
+	OP_NUMNOTEQUAL,
+	OP_LESSTHAN,
+	OP_GREATERTHAN,
+	OP_LESSTHANOREQUAL,
+	OP_GREATERTHANOREQUAL,
+	OP_MIN,
+	OP_MAX,
+
+	OP_WITHIN,
+
+	OP_RIPEMD160,
+	OP_SHA1,
+	OP_SHA256,
+	OP_HASH160,
+	OP_HASH256,
+	OP_CODESEPARATOR,
+	OP_CHECKSIG,
+	OP_CHECKSIGVERIFY,
+	OP_CHECKMULTISIG,
+	OP_CHECKMULTISIGVERIFY,
+
+	OP_NOP1,
+	OP_NOP2,
+	OP_NOP3,
+	OP_NOP4,
+	OP_NOP5,
+	OP_NOP6,
+	OP_NOP7,
+	OP_NOP8,
+	OP_NOP9,
+	OP_NOP10,
+
+	OP_SMALLINTEGER,
+	OP_PUBKEYS,
+	OP_PUBKEYHASH,
+	OP_PUBKEY,
 }
+
 OPCODE_NAMES = {
 	OP_0 : 'OP_0',
 	OP_PUSHDATA1 : 'OP_PUSHDATA1',
diff --git a/bitcoin/scripteval.py b/bitcoin/scripteval.py
@@ -141,14 +141,14 @@ def dumpstack(msg, stack):
 		print "#%d: %s" % (i, vch.encode('hex'))
 
 ISA_UNOP = {
-	OP_1ADD: True,
-	OP_1SUB: True,
-	OP_2MUL: True,
-	OP_2DIV: True,
-	OP_NEGATE: True,
-	OP_ABS: True,
-	OP_NOT: True,
-	OP_0NOTEQUAL: True,
+	OP_1ADD,
+	OP_1SUB,
+	OP_2MUL,
+	OP_2DIV,
+	OP_NEGATE,
+	OP_ABS,
+	OP_NOT,
+	OP_0NOTEQUAL,
 }
 
 def UnaryOp(opcode, stack):
@@ -189,21 +189,21 @@ def UnaryOp(opcode, stack):
 	return True
 
 ISA_BINOP = {
-	OP_ADD: True,
-	OP_SUB: True,
-	OP_LSHIFT: True,
-	OP_RSHIFT: True,
-	OP_BOOLAND: True,
-	OP_BOOLOR: True,
-	OP_NUMEQUAL: True,
-	OP_NUMEQUALVERIFY: True,
-	OP_NUMNOTEQUAL: True,
-	OP_LESSTHAN: True,
-	OP_GREATERTHAN: True,
-	OP_LESSTHANOREQUAL: True,
-	OP_GREATERTHANOREQUAL: True,
-	OP_MIN: True,
-	OP_MAX: True,
+	OP_ADD,
+	OP_SUB,
+	OP_LSHIFT,
+	OP_RSHIFT,
+	OP_BOOLAND,
+	OP_BOOLOR,
+	OP_NUMEQUAL,
+	OP_NUMEQUALVERIFY,
+	OP_NUMNOTEQUAL,
+	OP_LESSTHAN,
+	OP_GREATERTHAN,
+	OP_LESSTHANOREQUAL,
+	OP_GREATERTHANOREQUAL,
+	OP_MIN,
+	OP_MAX,
 }
 
 def BinOp(opcode, stack):
diff --git a/node.py b/node.py
@@ -44,10 +44,10 @@ def verbose_sendmsg(message):
 
 def verbose_recvmsg(message):
 	skipmsg = {
-		'tx' : True,
-		'block' : True,
-		'inv' : True,
-		'addr' : True
+		'tx',
+		'block',
+		'inv',
+		'addr',
 	}
 	if debugnet:
 		return True
diff --git a/rpc.py b/rpc.py
@@ -16,14 +16,14 @@
 import bitcoin.coredefs
 
 VALID_RPCS = {
-	"getblockcount" : True,
-	"getblockhash" : True,
-	"getconnectioncount" : True,
-	"getinfo" : True,
-	"getrawmempool" : True,
-	"getrawtransaction" : True,
-	"help" : True,
-	"stop" : True,
+	"getblockcount",
+	"getblockhash",
+	"getconnectioncount",
+	"getinfo",
+	"getrawmempool",
+	"getrawtransaction",
+	"help",
+	"stop",
 }
 
 
