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 vinay.sajip
Recipients markb, python-dev, serhiy.storchaka, vinay.sajip
Date 2017-01-11.09:40:23
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1484127623.82.0.302601086874.issue29220@psf.upfronthosting.co.za>
In-reply-to
Content
> I think that even with empty name the output contains extra separators

Not sure what you mean - please clarify with an example.

> I would add an indentation for the second "if" ... or even use multiple returns for the sake of microoptimization

I will do a further update soon.

> I don't understand the use of _nameToLevel. getLevelName('CRITICAL') returns 50, that even is not a string.

Historically, getLevelName() gets the name for a level *but also the level for a name* (I perhaps should have done a separate API for that, like getLevelForName(), but it's too late now due to backward compatibility requirements - it's been like this for a long time.)
History
Date User Action Args
2017-01-11 09:40:23vinay.sajipsetrecipients: + vinay.sajip, markb, python-dev, serhiy.storchaka
2017-01-11 09:40:23vinay.sajipsetmessageid: <1484127623.82.0.302601086874.issue29220@psf.upfronthosting.co.za>
2017-01-11 09:40:23vinay.sajiplinkissue29220 messages
2017-01-11 09:40:23vinay.sajipcreate