[issue42099] Fix reference to ob_type in unionobject.c and ceval

[Mark Shannon]

Mark Shannon  added the comment:

Thanks Neil

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

[issue42099] Fix reference to ob_type in unionobject.c and ceval

[Mark Shannon]

Mark Shannon  added the comment:


New changeset 0564aafb71a153dd0aca4b9266dfae9336a4f2cb by Neil Schemenauer in 
branch 'master':
bpo-42099: Fix reference to ob_type in unionobject.c and ceval (GH-22829)
https://github.com/python/cpython/commit/0564aafb71a153dd0aca4b9266dfae9336a4f2cb


--
nosy: +Mark.Shannon

___
Python tracker 

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

[issue42099] Fix reference to ob_type in unionobject.c and ceval

[Neil Schemenauer]

Change by Neil Schemenauer :


--
keywords: +patch
pull_requests: +21784
pull_request: https://github.com/python/cpython/pull/22829

___
Python tracker 

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

[issue42099] Fix reference to ob_type in unionobject.c and ceval

[Neil Schemenauer]

New submission from Neil Schemenauer :

It is great that access to ob_type has been cleaned up to use an access macro.  
There are two spots that still need fixing.

I think we should do something to help avoid this kind of thing slipping into 
the code in the future.  E.g. a special build flag that renames the ob_type 
member would work.  I'll make a separate bug for that.

--
assignee: vstinner
components: Interpreter Core
messages: 379155
nosy: nascheme, vstinner
priority: normal
severity: normal
stage: patch review
status: open
title: Fix reference to ob_type in unionobject.c and ceval
type: enhancement

___
Python tracker 

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