[issue44477] Remove ma_version_tag in dict?

[Ken Jin]

Ken Jin  added the comment:

Closing this for now as it seems PEP 509 may need to be repealed:
https://github.com/faster-cpython/ideas/issues/30#issuecomment-858615678

Sorry for the noise everyone :(.

--
resolution:  -> postponed
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

[issue44477] Remove ma_version_tag in dict?

[Ken Jin]

Ken Jin  added the comment:

Hmm I'm suddenly having second thoughts. I wonder if there's future use cases I 
might be missing here from PEP 509?

--
nosy: +vstinner

___
Python tracker 

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

[issue44477] Remove ma_version_tag in dict?

[Ken Jin]

Change by Ken Jin :


--
keywords: +patch
pull_requests: +25414
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/26833

___
Python tracker 

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

[issue44477] Remove ma_version_tag in dict?

[Ken Jin]

New submission from Ken Jin :

issue44206, issue44337, and issue44338 converted things dependent on 
ma_version_tag to ma_keys->dk_version.

I propose to remove ma_version_tag and save 8 bytes of space in dict. I've 
prepared a draft PR to do that. However, I'd like to wait for some time before 
merging it. This will make it easier to revert a PR in one of those issues 
should the need arise.

--
messages: 396264
nosy: Mark.Shannon, kj, methane
priority: normal
severity: normal
status: open
title: Remove ma_version_tag in dict?
versions: Python 3.11

___
Python tracker 

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