[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[Gregory P. Smith]

Gregory P. Smith  added the comment:

Thanks for the contribution!

(our attention spans can be scattered and random, apologies for the delay)

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[Gregory P. Smith]

Gregory P. Smith  added the comment:


New changeset dc2d8404a3ab6288ce112c71da8c65c34cd3087e by Dave Goncalves in 
branch 'main':
bpo-33178: Add BigEndianUnion, LittleEndianUnion classes to ctypes (GH-25480)
https://github.com/python/cpython/commit/dc2d8404a3ab6288ce112c71da8c65c34cd3087e


--

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[Gregory P. Smith]

Change by Gregory P. Smith :


--
assignee:  -> gregory.p.smith
nosy: +gregory.p.smith
versions: +Python 3.11 -Python 2.7, Python 3.6

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[David Goncalves]

David Goncalves  added the comment:

Any core developers available to review this PR?

--

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[David Goncalves]

David Goncalves  added the comment:

PR 25480 passes checks and awaits core review. All comments thus far have been 
addressed.

--

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[David Goncalves]

Change by David Goncalves :


--
keywords: +patch
nosy: +dpg
nosy_count: 6.0 -> 7.0
pull_requests: +24204
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/25480

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[Tomas Hyland]

Tomas Hyland  added the comment:

Hi there, I'm taking part in 'EnHackathon' spending several days over the next 
few weeks working on issues. I think this could be a good issue for a small 
team of first-time contributors with experience in C to work on.
Would anyone be able to offer guidance if we encounter problems?

--
nosy: +tohyland

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[emezh]

emezh  added the comment:

updated nosy list

--
nosy: +amaury.forgeotdarc, belopolsky, meador.inge, theller

___
Python tracker 

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

[issue33178] Add support for BigEndianUnion and LittleEndianUnion in ctypes

[emezh]

New submission from emezh :

Python documentation says that "To build structures with non-native byte order, 
you can use one of the BigEndianStructure, LittleEndianStructure, 
BigEndianUnion, and LittleEndianUnion base classes"

However, BigEndianUnion ad LittleEndianUnion are not implemented

>>> from ctypes import *
>>> BigEndianStructure

>>> BigEndianUnion
Traceback (most recent call last):
  File "", line 1, in 
NameError: name 'BigEndianUnion' is not defined

Is that something that can be added?

See also https://bugs.python.org/issue19023

--
components: ctypes
messages: 314647
nosy: Eugene Mezhibovsky
priority: normal
severity: normal
status: open
title: Add support for BigEndianUnion and LittleEndianUnion in ctypes
type: enhancement
versions: Python 2.7, Python 3.6

___
Python tracker 

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