Fix 0 length bencoded strings

Author: SevereOverfl0w

nrepl/bencode.py

@@ -46,7 +46,7 @@ def _read_int(s, terminator=None, init_data=None):
 
 
 def _read_bytes(s, n):
-    data = StringIO()
+    data = StringIO('')
     cnt = 0
     while cnt < n:
         m = s.read(n - cnt)
@@ -95,7 +95,7 @@ def _read_map(s):
 
 def _read_datum(s):
     delim = _read_delimiter(s)
-    if delim:
+    if delim is not '':
         return _read_fns.get(delim, lambda s: _read_bytes(s, delim))(s)
 
 
@@ -135,7 +135,7 @@ def encode(v):
 def decode_file(file):
     while True:
         x = _read_datum(file)
-        if not x:
+        if x is None:
             break
         yield x
 

test.py

@@ -23,6 +23,12 @@ def test_encoding (self):
         self.assertEqual([{u'a': 1, u'c': [{u'x': [u'y']}], u'b': [2, [3]]}],
                 list(decode('d1:ai1e1:cld1:xl1:yeee1:bli2eli3eeee')))
 
+    def test_empty_string (self):
+        self.assertEqual(['a'], list(decode('1:a')))
+        self.assertEqual([''], list(decode('0:')))
+        self.assertEqual([['spam', '', 'a', 'ab']], list(decode('l4:spam0:1:a2:abe')))
+        self.assertEqual([{'spam': ''}], list(decode('d4:spam0:e')))
+
 class REPLTest (unittest.TestCase):
     def setUp (self):
         # this here only to accommodate travis, which puts leiningen @ lein2