[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[INADA Naoki]

INADA Naoki  added the comment:


New changeset 0897e0c597c065f043e4286d01f16f473ab664ee by Inada Naoki in branch 
'master':
bpo-33504: fix wrong "versionchanged" (GH-11712)
https://github.com/python/cpython/commit/0897e0c597c065f043e4286d01f16f473ab664ee


--
nosy: +inada.naoki

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[INADA Naoki]

Change by INADA Naoki :


--
pull_requests: +11570, 11571, 11572

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[INADA Naoki]

Change by INADA Naoki :


--
pull_requests: +11570

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[INADA Naoki]

Change by INADA Naoki :


--
pull_requests: +11570, 11571

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[Łukasz Langa]

Change by Łukasz Langa :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[Łukasz Langa]

Łukasz Langa  added the comment:


New changeset 3a5b0d8988491d9408b22bceea6fd70b91345724 by Łukasz Langa (John 
Reese) in branch 'master':
bpo-33504: Migrate configparser from OrderedDict to dict. (#6819)
https://github.com/python/cpython/commit/3a5b0d8988491d9408b22bceea6fd70b91345724


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[Łukasz Langa]

Łukasz Langa  added the comment:

I am +1 to changing it to dict for Python 3.8.

This use case is different from `namedtuple._asdict`. The type of the 
dictionary is for the purpose of internal storage of the parser. Unlike 
`namedtuple._asdict`, it is not exposed to the user.

The purpose of changing the default to OrderedDict in the past is not 
documented but implying from its properties we can deduce it was to make 
minimal edits to .ini files stable. This property will remain unchanged with 
changing the type back to `dict`.

The history here is this:
- a `dict_type` was introduced by MvL in r52908 (issue 1371075) with the 
motivation to allow sorting or order preservation (December 2006);
- the default `dict_type` was changed by Raymond to OrderedDict for 3.1 in 
0663a1ed793c164fb11d3dd62bebc677e260891e (March 2009); as far as I can tell 
there was no issue for it when that change was made.

I acknowledge that in https://bugs.python.org/issue32360 Raymond states that 
"can't just change the default argument for configparser" but I don't see why. 
ConfigParser doesn't return the dictionary to the user in any API and the new 
3.6+ dictionary preserves order just as OrderedDict.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[John Reese]

Change by John Reese :


--
nosy: +lukasz.langa

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

See msg308974 in issue32360.

--
nosy: +rhettinger, serhiy.storchaka

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[John Reese]

Change by John Reese :


--
keywords: +patch
pull_requests: +6504
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[Eric V. Smith]

Eric V. Smith  added the comment:

Like #33463, it will require a deprecation cycle. I'm not sure it's worth the 
hassle, but I wouldn't oppose it if someone wanted to do the work. 3.8 would be 
the first version where it could be added.

--
nosy: +eric.smith
versions:  -Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue33504] configparser should use dict instead of OrderedDict in 3.7+

[John Reese]

New submission from John Reese :

The configparser module uses `collections.OrderedDict` as its default, but this 
is no longer necessary in 3.7+ due to the semantics of core dictionaries being 
ordered by design.  configparser should just use `dict` by default now, instead.

--
components: Library (Lib)
messages: 316547
nosy: jreese
priority: normal
severity: normal
status: open
title: configparser should use dict instead of OrderedDict in 3.7+
type: enhancement
versions: Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com