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
redundant "base" field in memoryview objects #47810
Comments
PyMemoryViewObject has a "base" field which points to the originator of Not removing "base" would make for a confusing and error-prone API. |
Why is this a "critical" bug? |
Because it should be fixed before 3.0 final? |
And why should that be done? IMO, this can still |
Le vendredi 15 août 2008 à 19:10 +0000, Martin v. Löwis a écrit :
I've filed this as critical because it is a new API and, if we change It is a minor redundancy but could easily lead to subtle problems Of course, if other people disagree, it can be bumped down to "normal". |
I don't think it is API. The structure may be defined in a public To make that clear, the structure could be moved into the C file, |
Ok, here is a simple patch. It:
I didn't try to clean up the existing documentation comments, although I Let me know what you think. |
I don't have time to review the patch, but based on the description, I'm |
Reviewers: GvR, Message: Description: Please review this at http://codereview.appspot.com/3003 Affected files: |
PS. The PEP probably needs an update. And the docs. |
On 2008/08/19 17:18:45, GvR wrote:
And Misc/NEWS |
Well, the PEP needs some love generally speaking; many things are un- or The patch is committed in r65862. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: