Skip to content

configparser.RawConfigParser._read is unmanageably complex#117348

@jaraco

Description

@jaraco

From the review:

This function is already unmanageably complex (and complexity checks disabled in the backport). Adding this single boolean parameter has expanded this function from ~120 lines to almost 160 and increases the mccabe cyclometric complexity of this function from 25 to 31 (where a target complexity is <10).

cpython main @ pip-run mccabe -- -m mccabe --min 14 Lib/configparser.py 940:4: 'RawConfigParser._read' 25 cpython main @ gh pr checkout 117273 Switched to branch 'unnamed-section' cpython unnamed-section @ pip-run mccabe -- -m mccabe --min 14 Lib/configparser.py 961:4: 'RawConfigParser._read' 31 

Originally posted by @jaraco in #117273 (comment)

Linked PRs

Metadata

Metadata

Assignees

Labels

stdlibStandard Library Python modules in the Lib/ directorytriagedThe issue has been accepted as valid by a triager.type-featureA feature request or enhancement

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions