Skip to content

gh-101819: Adapt _io types to heap types, batch 1 #101949

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.

Sign up for GitHub

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

Merged
miss-islington merged 20 commits into from
Feb 20, 2023
Merged

gh-101819: Adapt _io types to heap types, batch 1 #101949

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
6d0a37b
Add tests
erlend-aasland Feb 12, 2023
bd19a20
StringIO type
erlend-aasland Feb 11, 2023
9af73fe
PyTextIOWrapper type
erlend-aasland Feb 11, 2023
4bd73bd
Move get_io_state() to main header
erlend-aasland Feb 11, 2023
8d96a7a
PyFileIO type
erlend-aasland Feb 11, 2023
97f382f
Buffered* types
erlend-aasland Feb 12, 2023
eeb5c48
PyBytesIO type
erlend-aasland Feb 12, 2023
3c7e203
Remove unneded global state call
erlend-aasland Feb 16, 2023
254bfaf
Avoid PyState_FindModule as far as possible for now
erlend-aasland Feb 16, 2023
b3db018
Fixup clinic_state for module level functions
erlend-aasland Feb 16, 2023
0570ae0
Regen clinic
erlend-aasland Feb 16, 2023
c1926a6
Visit and clear heap types at module level
erlend-aasland Feb 16, 2023
471ae9f
Fix tests on Windows
erlend-aasland Feb 16, 2023
345ffd3
Pull in main
erlend-aasland Feb 16, 2023
65c0086
Fix unused variable warning
erlend-aasland Feb 16, 2023
39e0a89
Merge branch 'main' into isolate-io/1-heap-types
erlend-aasland Feb 17, 2023
86a6dbc
Merge branch 'main' into isolate-io/1-heap-types
kumaraditya303 Feb 19, 2023
cf1eb47
Merge branch 'main' into isolate-io/1-heap-types
erlend-aasland Feb 19, 2023
7e0f327
Pull in main
erlend-aasland Feb 20, 2023
cf605e6
Use Py_DEBUG iso. NDEBUG
erlend-aasland Feb 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Move get_io_state() to main header
  • Loading branch information
@erlend-aasland
erlend-aasland committed Feb 15, 2023
commit 4bd73bdd2753b40122ff09ff67476c03d785a1cd
9 changes: 0 additions & 9 deletions Modules/_io/_iomodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
#define PY_SSIZE_T_CLEAN
#include "Python.h"
#include "_iomodule.h"
#include "pycore_moduleobject.h" // _PyModule_GetState()
#include "pycore_pystate.h" // _PyInterpreterState_GET()

#ifdef HAVE_SYS_TYPES_H
Expand Down Expand Up @@ -559,14 +558,6 @@ PyNumber_AsOff_t(PyObject *item, PyObject *err)
return result;
}

static inline _PyIO_State*
get_io_state(PyObject *module)
{
void *state = _PyModule_GetState(module);
assert(state != NULL);
return (_PyIO_State *)state;
}

_PyIO_State *
_PyIO_get_module_state(void)
{
Expand Down
9 changes: 9 additions & 0 deletions Modules/_io/_iomodule.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

#include "exports.h"

#include "pycore_moduleobject.h" // _PyModule_GetState()
#include "structmember.h"

/* ABCs */
Expand Down Expand Up @@ -153,6 +154,14 @@ typedef struct {
#define IO_MOD_STATE(mod) ((_PyIO_State *)PyModule_GetState(mod))
#define IO_STATE() _PyIO_get_module_state()

static inline _PyIO_State *
get_io_state(PyObject *module)
{
void *state = _PyModule_GetState(module);
assert(state != NULL);
return (_PyIO_State *)state;
}

extern _PyIO_State *_PyIO_get_module_state(void);

#ifdef MS_WINDOWS
Expand Down