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