Message285204
> 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.) |
|
Date |
User |
Action |
Args |
2017-01-11 09:40:23 | vinay.sajip | set | recipients:
+ vinay.sajip, markb, python-dev, serhiy.storchaka |
2017-01-11 09:40:23 | vinay.sajip | set | messageid: <1484127623.82.0.302601086874.issue29220@psf.upfronthosting.co.za> |
2017-01-11 09:40:23 | vinay.sajip | link | issue29220 messages |
2017-01-11 09:40:23 | vinay.sajip | create | |
|