mirror of
https://github.com/python/cpython.git
synced 2025-10-19 05:08:28 +00:00
Get rid of builtin_open() entirely (the C code and docstring, not the
builtin function); Guido pointed out that it could be just another name in the __builtin__ dict for the file constructor now.
This commit is contained in:
parent
4b7625ee83
commit
742dfd6f17
2 changed files with 5 additions and 22 deletions
|
@ -1359,7 +1359,6 @@ file_new(PyTypeObject *type, PyObject *args, PyObject *kwds)
|
||||||
return f;
|
return f;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* XXX Keep this in synch with open_doc in bltinmodule.c. */
|
|
||||||
static char file_doc[] =
|
static char file_doc[] =
|
||||||
"file(name[, mode[, buffering]]) -> file object\n"
|
"file(name[, mode[, buffering]]) -> file object\n"
|
||||||
"\n"
|
"\n"
|
||||||
|
@ -1369,7 +1368,8 @@ static char file_doc[] =
|
||||||
"opened for writing. Add a 'b' to the mode for binary files.\n"
|
"opened for writing. Add a 'b' to the mode for binary files.\n"
|
||||||
"Add a '+' to the mode to allow simultaneous reading and writing.\n"
|
"Add a '+' to the mode to allow simultaneous reading and writing.\n"
|
||||||
"If the buffering argument is given, 0 means unbuffered, 1 means line\n"
|
"If the buffering argument is given, 0 means unbuffered, 1 means line\n"
|
||||||
"buffered, and larger numbers specify the buffer size.";
|
"buffered, and larger numbers specify the buffer size.\n"
|
||||||
|
"Note: open() is an alias for file().\n";
|
||||||
|
|
||||||
PyTypeObject PyFile_Type = {
|
PyTypeObject PyFile_Type = {
|
||||||
PyObject_HEAD_INIT(&PyType_Type)
|
PyObject_HEAD_INIT(&PyType_Type)
|
||||||
|
|
|
@ -1189,25 +1189,6 @@ static char oct_doc[] =
|
||||||
Return the octal representation of an integer or long integer.";
|
Return the octal representation of an integer or long integer.";
|
||||||
|
|
||||||
|
|
||||||
static PyObject *
|
|
||||||
builtin_open(PyObject *self, PyObject *args)
|
|
||||||
{
|
|
||||||
return PyFile_Type.tp_new(&PyFile_Type, args, NULL);
|
|
||||||
}
|
|
||||||
|
|
||||||
/* XXX Keep this in synch with file_doc in fileobject.c. */
|
|
||||||
static char open_doc[] =
|
|
||||||
"open(name[, mode[, buffering]]) -> file object\n"
|
|
||||||
"\n"
|
|
||||||
"Open a file. The mode can be 'r', 'w' or 'a' for reading (default),\n"
|
|
||||||
"writing or appending. The file will be created if it doesn't exist\n"
|
|
||||||
"when opened for writing or appending; it will be truncated when\n"
|
|
||||||
"opened for writing. Add a 'b' to the mode for binary files.\n"
|
|
||||||
"Add a '+' to the mode to allow simultaneous reading and writing.\n"
|
|
||||||
"If the buffering argument is given, 0 means unbuffered, 1 means line\n"
|
|
||||||
"buffered, and larger numbers specify the buffer size.";
|
|
||||||
|
|
||||||
|
|
||||||
static PyObject *
|
static PyObject *
|
||||||
builtin_ord(PyObject *self, PyObject* obj)
|
builtin_ord(PyObject *self, PyObject* obj)
|
||||||
{
|
{
|
||||||
|
@ -1802,7 +1783,6 @@ static PyMethodDef builtin_methods[] = {
|
||||||
{"max", builtin_max, METH_VARARGS, max_doc},
|
{"max", builtin_max, METH_VARARGS, max_doc},
|
||||||
{"min", builtin_min, METH_VARARGS, min_doc},
|
{"min", builtin_min, METH_VARARGS, min_doc},
|
||||||
{"oct", builtin_oct, METH_O, oct_doc},
|
{"oct", builtin_oct, METH_O, oct_doc},
|
||||||
{"open", builtin_open, METH_VARARGS, open_doc},
|
|
||||||
{"ord", builtin_ord, METH_O, ord_doc},
|
{"ord", builtin_ord, METH_O, ord_doc},
|
||||||
{"pow", builtin_pow, METH_VARARGS, pow_doc},
|
{"pow", builtin_pow, METH_VARARGS, pow_doc},
|
||||||
{"range", builtin_range, METH_VARARGS, range_doc},
|
{"range", builtin_range, METH_VARARGS, range_doc},
|
||||||
|
@ -1861,6 +1841,9 @@ _PyBuiltin_Init(void)
|
||||||
SETBUILTIN("super", &PySuper_Type);
|
SETBUILTIN("super", &PySuper_Type);
|
||||||
SETBUILTIN("tuple", &PyTuple_Type);
|
SETBUILTIN("tuple", &PyTuple_Type);
|
||||||
SETBUILTIN("type", &PyType_Type);
|
SETBUILTIN("type", &PyType_Type);
|
||||||
|
|
||||||
|
/* Note that open() is just an alias of file(). */
|
||||||
|
SETBUILTIN("open", &PyFile_Type);
|
||||||
SETBUILTIN("file", &PyFile_Type);
|
SETBUILTIN("file", &PyFile_Type);
|
||||||
#ifdef Py_USING_UNICODE
|
#ifdef Py_USING_UNICODE
|
||||||
SETBUILTIN("unicode", &PyUnicode_Type);
|
SETBUILTIN("unicode", &PyUnicode_Type);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue