[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

If the capability of the former plistlib.Dict class is useful, it would be 
better to provide it for general use in modules collections or types.

--
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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Serhiy Storchaka added the comment:


New changeset edef358ed6d05f927bf1636cc5a920a9d868b131 by Serhiy Storchaka in 
branch 'master':
bpo-29196: Removed old-deprecated classes Plist, Dict and _InternalDict (#488)
https://github.com/python/cpython/commit/edef358ed6d05f927bf1636cc5a920a9d868b131


--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Ronald Oussoren]

Ronald Oussoren added the comment:

I don't know why the feature was deprecated, is was deprecated a long time ago 
by Just.

I agree that the feature is convenient at times, but it is also different from 
how dicts are generally used. 

BTW. I'm in favour of removing the feature, not having this deprecated feature 
makes the library easier to understand and maintain.

P.S. plistlib was not introduced in 2.6, but in 2.3. In 2.3 it was part of the 
MacOS library (Lib/plat-mac) and in 2.6 it was migrated to the regular stdlib.

--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Raymond Hettinger]

Raymond Hettinger added the comment:

Too bad the checkin message didn't say why the feature was deprecated.  

That said, if anyone cared, they would have spoken-up by now.

--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

The deprecation was added in 368c0b22f8e43709afbbc3805f55ceffed127212. The 
comment added in 4c3d054d3d3fa693e4eff4574ab8e98772bd66cd said that Dict was 
kept for compatibility with Python 2.2.

--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Raymond Hettinger]

Raymond Hettinger added the comment:

Ronald, I'm curious about why was the attribute access capability was ever 
deprecated.  That capability seemed nice to have.

--
nosy: +rhettinger

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

What are your thoughts about this Ronald?

--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

These classes was deprecated in 2.4. In 2.6 the plistlib module was moved from 
plat-mac to the general stdlib library.

--

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
pull_requests: +401

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
assignee:  -> ronaldoussoren

___
Python tracker 

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

[issue29196] Remove old-deprecated plistlib features

[Serhiy Storchaka]

New submission from Serhiy Storchaka:

Seems many features of plistlib was deprecated from the start. The plistlib 
module was added in 2.6 and its code already contained deprecated features:

* The _InternalDict class is a dict subclass with implemented 
__getattr__/__setattr__/__delattr__ methods, but all these methods have been 
deprecated. Since deprecated methods shouldn't be used and they are not 
implemented in builting dict, _InternalDict can be replaced by builting dict.

* The Dict class has been deprecated. It doesn't used in the module. A user 
should use builting dict instead.

* The Plist class has been deprecated. It doesn't used in the module. A user 
should use module level functions instead of Plist methods.

It seems to me that all these classes can be removed. This doesn't break 
compatibility, since they were deprecated in 2.6. They are even not documented 
in 2.7.

--
components: Library (Lib)
files: plistlib_remove_deprecated_classes.patch
keywords: patch
messages: 284925
nosy: ronaldoussoren, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Remove old-deprecated plistlib features
type: enhancement
versions: Python 3.7
Added file: 
http://bugs.python.org/file46196/plistlib_remove_deprecated_classes.patch

___
Python tracker 

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