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
Prev Previous commit
Next Next commit
Refactor: hide delegate 
Add method wraper for _PyIOBase_check_readable
  • Loading branch information
@erlend-aasland
erlend-aasland committed May 9, 2023
commit 703440a7e1717d518ac5b848e6fe72dc002d8bf1
8 changes: 7 additions & 1 deletion Modules/_io/iobase.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,12 @@ iobase_check_seekable(PyObject *self, PyObject *args)
return _PyIOBase_check_seekable(self, args);
}

static PyObject *
iobase_check_readable(PyObject *self, PyObject *args)
{
return _PyIOBase_check_readable(self, args);
}

/* XXX: IOBase thinks it has to maintain its own internal state in
`__IOBase_closed` and call flush() by itself, but it is redundant with
whatever behaviour a non-trivial derived class will implement. */
Expand Down Expand Up @@ -809,7 +815,7 @@ static PyMethodDef iobase_methods[] = {

{"_checkClosed", _PyIOBase_check_closed, METH_NOARGS},
{"_checkSeekable", iobase_check_seekable, METH_NOARGS},
{"_checkReadable", _PyIOBase_check_readable, METH_NOARGS},
{"_checkReadable", iobase_check_readable, METH_NOARGS},
{"_checkWritable", _PyIOBase_check_writable, METH_NOARGS},

_IO__IOBASE_FILENO_METHODDEF
Expand Down