decode_long(): Simplified the "is it negative?" test.

80ac79bc · Tim Peters · b828cd7a · 80ac79bc
Commit 80ac79bc authored Jan 27, 2003 by Tim Peters
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

Lib/pickletools.py Lib/pickletools.py +5 -2

No files found.
--- a/Lib/pickletools.py
+++ b/Lib/pickletools.py
@@ -590,14 +590,17 @@ def decode_long(data):
    -256L
    >>> decode_long("\x00\x80")
    -32768L
-    >>> 
+    >>> decode_long("\x80")
+    -128L
+    >>> decode_long("\x7f")
+    127L
    """
    x = 0L
    i = 0L
    for c in data:
        x |= long(ord(c)) << i
        i += 8L
-    if i and (x & (1L << (i-1L))):
+    if data and ord(c) >= 0x80:
        x -= 1L << i
    return x