This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author berker.peksag
Recipients barry, berker.peksag, durin42, gward, ncoghlan, pitrou, r.david.murray, terry.reedy
Date 2015-04-15.14:36:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
Some small comments:

* diff_bytes needs to be documented in the difflib docs and in Doc/whatsnew/3.5.rst.

* diff_bytes needs to be added to difflib.__all__

* This looks like a new feature to me, so it would be better to just commit it to the default branch.

+        except AttributeError:
+            raise TypeError('all arguments must be bytes, not %r' % s)

This could be changed to raise TypeError(...) from None

+            self.assertTrue(
+                isinstance(line, bytes),


+        try:
+            list(difflib.unified_diff(a, b, fna, fnb))
+  'expected TypeError')
+        except TypeError:
+            pass

with self.assertRaises(TypeError):
    list(difflib.unified_diff(a, b, fna, fnb))

looks more readable to me.
Date User Action Args
2015-04-15 14:36:18berker.peksagsetrecipients: + berker.peksag, barry, gward, terry.reedy, ncoghlan, pitrou, durin42, r.david.murray
2015-04-15 14:36:18berker.peksagsetmessageid: <>
2015-04-15 14:36:18berker.peksaglinkissue17445 messages
2015-04-15 14:36:18berker.peksagcreate