Make bitcoin.core.bignum internal use

May be replaced, as was done in Bitcoin Core

 [ yapified by gitreformat (github/ghtdak) on Mon Nov 30 21:11:37 2015 ]

Author: petertodd

README

@@ -26,7 +26,6 @@ consensus critical and non-consensus-critical.
 
 bitcoin.core            - Basic core definitions, datastructures, and
                           (context-independent) validation
-bitcoin.core.bignum     - Bignum handling
 bitcoin.core.key        - ECC pubkeys
 bitcoin.core.script     - Scripts and opcodes
 bitcoin.core.scripteval - Script evaluation/verification

bitcoin/core/bignum.py bitcoin/core/_bignum.pybitcoin/core/bignum.py renamed to bitcoin/core/_bignum.py

@@ -8,7 +8,10 @@
 # No part of python-bitcoinlib, including this file, may be copied, modified,
 # propagated, or distributed except according to the terms contained in the
 # LICENSE file.
-"""Bignum routines"""
+
+# Bignum routines
+#
+# Internally used for script evaluation; not to be used externally.
 
 from __future__ import absolute_import, division, print_function, unicode_literals
 

bitcoin/core/script.py

@@ -28,7 +28,7 @@
 import struct
 
 import bitcoin.core
-import bitcoin.core.bignum
+import bitcoin.core._bignum
 
 MAX_SCRIPT_SIZE = 10000
 MAX_SCRIPT_ELEMENT_SIZE = 520
@@ -527,8 +527,8 @@ def __coerce_instance(cls, other):
             elif other == -1:
                 other = bytes(_bchr(OP_1NEGATE))
             else:
-                other = CScriptOp.encode_op_pushdata(bitcoin.core.bignum.bn2vch(
-                    other))
+                other = CScriptOp.encode_op_pushdata(
+                    bitcoin.core._bignum.bn2vch(other))
         elif isinstance(other, (bytes, bytearray)):
             other = CScriptOp.encode_op_pushdata(other)
         return other

bitcoin/core/scripteval.py

@@ -27,6 +27,7 @@
 import hashlib
 
 import bitcoin.core
+import bitcoin.core._bignum
 import bitcoin.core.key
 import bitcoin.core.serialize
 
@@ -113,7 +114,7 @@ def __init__(self, opcode, **kwargs):
 
 
 def _CastToBigNum(s, err_raiser):
-    v = bitcoin.core.bignum.vch2bn(s)
+    v = bitcoin.core._bignum.vch2bn(s)
     if len(s) > MAX_NUM_SIZE:
         raise err_raiser(EvalScriptError, 'CastToBigNum() : overflow')
     return v
@@ -252,7 +253,7 @@ def _UnaryOp(opcode, stack, err_raiser):
         raise AssertionError(
             "Unknown unary opcode encountered; this should not happen")
 
-    stack.append(bitcoin.core.bignum.bn2vch(bn))
+    stack.append(bitcoin.core._bignum.bn2vch(bn))
 
 # OP_LSHIFT and OP_RSHIFT are *not* included in this list as they are disabled
 _ISA_BINOP = {
@@ -340,7 +341,7 @@ def _BinOp(opcode, stack, err_raiser):
 
     stack.pop()
     stack.pop()
-    stack.append(bitcoin.core.bignum.bn2vch(bn))
+    stack.append(bitcoin.core._bignum.bn2vch(bn))
 
 
 def _CheckExec(vfExec):
@@ -419,7 +420,7 @@ def check_args(n):
 
             if sop == OP_1NEGATE or ((sop >= OP_1) and (sop <= OP_16)):
                 v = sop - (OP_1 - 1)
-                stack.append(bitcoin.core.bignum.bn2vch(v))
+                stack.append(bitcoin.core._bignum.bn2vch(v))
 
             elif sop in _ISA_BINOP:
                 _BinOp(sop, stack, err_raiser)
@@ -511,7 +512,7 @@ def check_args(n):
 
             elif sop == OP_DEPTH:
                 bn = len(stack)
-                stack.append(bitcoin.core.bignum.bn2vch(bn))
+                stack.append(bitcoin.core._bignum.bn2vch(bn))
 
             elif sop == OP_DROP:
                 check_args(1)
@@ -631,7 +632,7 @@ def check_args(n):
             elif sop == OP_SIZE:
                 check_args(1)
                 bn = len(stack[-1])
-                stack.append(bitcoin.core.bignum.bn2vch(bn))
+                stack.append(bitcoin.core._bignum.bn2vch(bn))
 
             elif sop == OP_SHA1:
                 check_args(1)