Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Let ctypes._CData.from_buffer[_copy] accept any buffer object #2610

Merged
merged 1 commit into from
Nov 20, 2018

Conversation

dgelessus
Copy link
Contributor

Fixes the issue discussed here: #1906 (comment)

@srittau
Copy link
Collaborator

srittau commented Nov 20, 2018

Relevant: python/typing#593.

@JelleZijlstra
Copy link
Member

This seems OK for now; we may have to tweak it if/when we add buffer support to typing.

@JelleZijlstra JelleZijlstra merged commit 45d4e88 into python:master Nov 20, 2018
yedpodtrzitko pushed a commit to yedpodtrzitko/typeshed that referenced this pull request Jan 23, 2019
hauntsaninja pushed a commit to hauntsaninja/typeshed that referenced this pull request Jan 30, 2020
Follows the approach in python#2610
hauntsaninja pushed a commit to hauntsaninja/typeshed that referenced this pull request Feb 5, 2020
Follows the approach in python#2610
JelleZijlstra pushed a commit that referenced this pull request Feb 22, 2020
* fcntl: mark positional-only args

* fcntl: use overload for fcntl.fcntl

The comment about depending on the type of arg seems incorrect
bf50135

I checked the docs and examples, CPython implementation and CPython tests, but
I might be missing something

* fcntl: use overload for fcntl.ioctl

Based off of docs and examples

* fcntl: type buffers better

Follows the approach in #2610
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants