Vajrasky Kok added the comment:
Anyway, I found another issue on line 393 on the same file:
def test_del_nonexistent_param(self):
msg = Message()
msg.add_header('Content-Type', 'text/plain', charset='utf-8')
existing_header = msg['Content-Type']
msg.del_param('foobar', header='Content-Type')
self.assertEqual(msg['Content-Type'], 'text/plain; charset="utf-8"')
The variable existing_header is never used. Either we can remove it or change
it to:
def test_del_nonexistent_param(self):
msg = Message()
msg.add_header('Content-Type', 'text/plain', charset='utf-8')
existing_header = msg['Content-Type']
msg.del_param('foobar', header='Content-Type')
self.assertEqual(msg['Content-Type'], existing_header)
At first, I wanted to create a ticket for this. But then, I thought why do not
we combine these menial problems together?
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue18503>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
