_loggerClass is initialized twice

[serhiy-storchaka]

BPO	23151
Nosy	@vsajip, @serhiy-storchaka

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2015-01-06.10:56:23.646>
created_at = <Date 2015-01-02.16:04:33.119>
labels = ['library']
title = '_loggerClass is initialized twice'
updated_at = <Date 2015-01-06.10:56:23.645>
user = 'https://github.com/serhiy-storchaka'

bugs.python.org fields:

activity = <Date 2015-01-06.10:56:23.645>
actor = 'python-dev'
assignee = 'none'
closed = True
closed_date = <Date 2015-01-06.10:56:23.646>
closer = 'python-dev'
components = ['Library (Lib)']
creation = <Date 2015-01-02.16:04:33.119>
creator = 'serhiy.storchaka'
dependencies = []
files = []
hgrepos = []
issue_num = 23151
keywords = []
message_count = 5.0
messages = ['233313', '233315', '233317', '233321', '233515']
nosy_count = 3.0
nosy_names = ['vinay.sajip', 'python-dev', 'serhiy.storchaka']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue23151'
versions = []

[serhiy-storchaka]

In Lib/logging/init.py at line 1089 _loggerClass is initialized to None. The code in Manager.getLogger() expects that it can be None. But at line 1549 _loggerClass is initialized to Logger. And there is no official way to set it to None. Looks as either initialization to None and the code in Manager.getLogger() are redundant or initialization to Logger is redundant.

[vsajip]

The code in Manager.getLogger() allows an overriding logger class for that manager instance only - if it's not set (which is the default), it uses the module global _loggerClass. The lines

rv = (self.loggerClass or _loggerClass)(name)

indicate this.

The module global _loggerClass is initialized to None initially, simply so that it can be defined before being used in Manager (so really, for readability). Later, after Logger is defined, _loggerClass is set to Logger.

Is this causing some actual problem?

[serhiy-storchaka]

Is this causing some actual problem?

No, this does not cause actual problem except that it looks confusing. So really this decreased readability to me.

[vsajip]

Okay, I'll see if I can make it clearer.

[python-dev]

New changeset 8bfe230db0bc by Vinay Sajip in branch 'default':
Closes bpo-23151: Removed unnecessary initialization.
https://hg.python.org/cpython/rev/8bfe230db0bc