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 vajrasky
Recipients r.david.murray, vajrasky
Date 2013-07-19.16:06:37
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1374249997.56.0.452353924918.issue18503@psf.upfronthosting.co.za>
In-reply-to
Content
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?
History
Date User Action Args
2013-07-19 16:06:37vajraskysetrecipients: + vajrasky, r.david.murray
2013-07-19 16:06:37vajraskysetmessageid: <1374249997.56.0.452353924918.issue18503@psf.upfronthosting.co.za>
2013-07-19 16:06:37vajraskylinkissue18503 messages
2013-07-19 16:06:37vajraskycreate