From: Benjamin Peterson <benjamin@python.org>
Date: Thu, 4 Sep 2008 02:22:52 +0000 (+0000)
Subject: #2501 xml.sax.parser doesn't terminate when given a filename; enable some more tests!
X-Git-Tag: v3.0rc1~108
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a7f4f5a772163322178ccb4c954e3bd04da5317f;p=python

#2501 xml.sax.parser doesn't terminate when given a filename; enable some more tests!

Reviewed by myself
---

diff --git a/Lib/test/test_sax.py b/Lib/test/test_sax.py
index 5f1a48ed48..a45d1f322e 100644
--- a/Lib/test/test_sax.py
+++ b/Lib/test/test_sax.py
@@ -446,8 +446,7 @@ class ExpatReaderTest(XmlTestBase):
 
     # ===== InputSource support
 
-    def XXXtest_expat_inpsource_filename(self):
-        # FIXME: test blocks indefinitely
+    def test_expat_inpsource_filename(self):
         parser = create_parser()
         result = StringIO()
         xmlgen = XMLGenerator(result)
@@ -457,8 +456,7 @@ class ExpatReaderTest(XmlTestBase):
 
         self.assertEquals(result.getvalue(), xml_test_out)
 
-    def XXXtest_expat_inpsource_sysid(self):
-        # FIXME: test blocks indefinitely
+    def test_expat_inpsource_sysid(self):
         parser = create_parser()
         result = StringIO()
         xmlgen = XMLGenerator(result)
@@ -531,8 +529,7 @@ class ExpatReaderTest(XmlTestBase):
         self.assertEquals(parser.getPublicId(), None)
         self.assertEquals(parser.getLineNumber(), 1)
 
-    def XXXtest_expat_locator_withinfo(self):
-        # FIXME: test blocks indefinitely
+    def test_expat_locator_withinfo(self):
         result = StringIO()
         xmlgen = XMLGenerator(result)
         parser = create_parser()
diff --git a/Lib/xml/sax/xmlreader.py b/Lib/xml/sax/xmlreader.py
index 6264ee3f01..5ce30b042a 100644
--- a/Lib/xml/sax/xmlreader.py
+++ b/Lib/xml/sax/xmlreader.py
@@ -119,7 +119,7 @@ class IncrementalParser(XMLReader):
         self.prepareParser(source)
         file = source.getByteStream()
         buffer = file.read(self._bufsize)
-        while buffer != "":
+        while buffer:
             self.feed(buffer)
             buffer = file.read(self._bufsize)
         self.close()