Vinay Sajip added the comment:

> 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.)

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue29220>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to