Skip to content

gh-101819: Refactor _io in preparation for module isolation #104334

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
erlend-aasland merged 12 commits into from
May 9, 2023
Merged

gh-101819: Refactor _io in preparation for module isolation #104334

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
14f923e
Refactor: replace query with parameter
erlend-aasland May 9, 2023
482f134
Refactor: replace query with parameter
erlend-aasland May 9, 2023
8adc0cd
Refactor: hide delegate
erlend-aasland May 9, 2023
703440a
Refactor: hide delegate
erlend-aasland May 9, 2023
ea9c098
Refactor: hide delegate
erlend-aasland May 9, 2023
e940ed3
Add io state struct forward declaration.
erlend-aasland May 9, 2023
c68d657
Refactor: replace query with parameter
erlend-aasland May 9, 2023
0084715
Refactor: replace query with parameter
erlend-aasland May 9, 2023
20777e7
Refactor: replace query with parameter
erlend-aasland May 9, 2023
b22b41d
Fetch state by module def in _io__Buffered_seek_impl()
erlend-aasland May 9, 2023
0044293
Address review: improve forward decl. of _PyIO_State
erlend-aasland May 9, 2023
7aa8da7
Address review: remove second typedef
erlend-aasland May 9, 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
Next Next commit
Refactor: replace query with parameter 
Pass module state to bufferediobase_unsupported()
  • Loading branch information
@erlend-aasland
erlend-aasland committed May 9, 2023
commit 14f923e938d89053653d3722770d81f8a5c814db
21 changes: 12 additions & 9 deletions Modules/_io/bufferedio.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,11 +106,9 @@ _io__BufferedIOBase_readinto1_impl(PyObject *self, Py_buffer *buffer)
}

static PyObject *
bufferediobase_unsupported(const char *message)
bufferediobase_unsupported(_PyIO_State *state, const char *message)
{
_PyIO_State *state = IO_STATE();
if (state != NULL)
PyErr_SetString(state->unsupported_operation, message);
PyErr_SetString(state->unsupported_operation, message);
return NULL;
}

Expand All @@ -127,7 +125,8 @@ static PyObject *
_io__BufferedIOBase_detach_impl(PyObject *self)
/*[clinic end generated code: output=754977c8d10ed88c input=822427fb58fe4169]*/
{
return bufferediobase_unsupported("detach");
_PyIO_State *state = IO_STATE();
return bufferediobase_unsupported(state, "detach");
}

PyDoc_STRVAR(bufferediobase_read_doc,
Expand All @@ -151,7 +150,8 @@ PyDoc_STRVAR(bufferediobase_read_doc,
static PyObject *
bufferediobase_read(PyObject *self, PyObject *args)
{
return bufferediobase_unsupported("read");
_PyIO_State *state = IO_STATE();
return bufferediobase_unsupported(state, "read");
}

PyDoc_STRVAR(bufferediobase_read1_doc,
Expand All @@ -164,7 +164,8 @@ PyDoc_STRVAR(bufferediobase_read1_doc,
static PyObject *
bufferediobase_read1(PyObject *self, PyObject *args)
{
return bufferediobase_unsupported("read1");
_PyIO_State *state = IO_STATE();
return bufferediobase_unsupported(state, "read1");
}

PyDoc_STRVAR(bufferediobase_write_doc,
Expand All @@ -179,7 +180,8 @@ PyDoc_STRVAR(bufferediobase_write_doc,
static PyObject *
bufferediobase_write(PyObject *self, PyObject *args)
{
return bufferediobase_unsupported("write");
_PyIO_State *state = IO_STATE();
return bufferediobase_unsupported(state, "write");
}


Expand Down Expand Up @@ -1298,7 +1300,8 @@ _io__Buffered_truncate_impl(buffered *self, PyObject *pos)
CHECK_INITIALIZED(self)
CHECK_CLOSED(self, "truncate of closed file")
if (!self->writable) {
return bufferediobase_unsupported("truncate");
_PyIO_State *state = IO_STATE();
return bufferediobase_unsupported(state, "truncate");
}
if (!ENTER_BUFFERED(self))
return NULL;
Expand Down