SF #1479181: split open() and file() from being aliases for each other.

diff --git a/Doc/lib/libfuncs.tex b/Doc/lib/libfuncs.tex
index 686db3957abb322dc2fa6ebf6db499a628c65c35..ff81faab182bdae0f2f4aec6ae6d628c22ddf143 100644 (file)
--- a/Doc/lib/libfuncs.tex
+++ b/Doc/lib/libfuncs.tex
@@ -455,12 +455,7 @@ class C:
     after any I/O has been performed, and there's no reliable way to
     determine whether this is the case.}
 
-  The \function{file()} constructor is new in Python 2.2 and is an
-  alias for \function{open()}.  Both spellings are equivalent.  The
-  intent is for \function{open()} to continue to be preferred for use
-  as a factory function which returns a new \class{file} object.  The
-  spelling, \class{file} is more suited to type testing (for example,
-  writing \samp{isinstance(f, file)}).
+  \versionadded{2.2}
 \end{funcdesc}
 
 \begin{funcdesc}{filter}{function, list}
@@ -725,7 +720,10 @@ class C:
 \end{funcdesc}
 
 \begin{funcdesc}{open}{filename\optional{, mode\optional{, bufsize}}}
-  An alias for the \function{file()} function above.
+  A wrapper for the \function{file()} function above.  The intent is
+  for \function{open()} to be preferred for use as a factory function
+  returning a new \class{file} object.  \class{file} is more suited to
+  type testing (for example, writing \samp{isinstance(f, file)}).
 \end{funcdesc}
 
 \begin{funcdesc}{ord}{c}

diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py
index c351ee92b13e48c9579dd49d89b62854b47a4d9f..88a4baa87decfba2c7b6452b0e422e62ce2dde0f 100644 (file)
--- a/Lib/test/test_subprocess.py
+++ b/Lib/test/test_subprocess.py
@@ -347,7 +347,7 @@ class ProcessTestCase(unittest.TestCase):
                          stdout=subprocess.PIPE,
                          universal_newlines=1)
         stdout = p.stdout.read()
-        if hasattr(open, 'newlines'):
+        if hasattr(p.stdout, 'newlines'):
             # Interpreter with universal newline support
             self.assertEqual(stdout,
                              "line1\nline2\nline3\nline4\nline5\nline6")
@@ -374,7 +374,7 @@ class ProcessTestCase(unittest.TestCase):
                          stdout=subprocess.PIPE, stderr=subprocess.PIPE,
                          universal_newlines=1)
         (stdout, stderr) = p.communicate()
-        if hasattr(open, 'newlines'):
+        if hasattr(stdout, 'newlines'):
             # Interpreter with universal newline support
             self.assertEqual(stdout,
                              "line1\nline2\nline3\nline4\nline5\nline6")

diff --git a/Misc/NEWS b/Misc/NEWS
index 1a180c29e3fc038ed6922449eb49d8b30fd2da58..8d88d9dc51728860a6214843dd0cc689b93c0d73 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,6 +12,8 @@ What's New in Python 2.5 alpha 2?
 Core and builtins
 -----------------
 
+- Patch #1479181: split open() and file() from being aliases for each other.
+
 - Bug #1465834: 'bdist_wininst preinstall script support' was fixed
   by converting these apis from macros into exported functions again:
 

diff --git a/Objects/fileobject.c b/Objects/fileobject.c
index e08afe64ba2fcb673374571cab7a0bf6dfe21384..25b336176685c15e340cd83dcc5b05de95c3d753 100644 (file)
--- a/Objects/fileobject.c
+++ b/Objects/fileobject.c
@@ -2025,10 +2025,6 @@ PyDoc_STR(
 "'\\r', '\\n', '\\r\\n' or a tuple containing all the newline types seen.\n"
 "\n"
 "'U' cannot be combined with 'w' or '+' mode.\n"
-)
-PyDoc_STR(
-"\n"
-"Note:  open() is an alias for file()."
 );
 
 PyTypeObject PyFile_Type = {

diff --git a/Python/bltinmodule.c b/Python/bltinmodule.c
index 27b481152ab3eaec250059b44a62b0a199a8607e..6fcc05e077803d93b1be3dbde8f1261b5e60a119 100644 (file)
--- a/Python/bltinmodule.c
+++ b/Python/bltinmodule.c
@@ -1341,6 +1341,18 @@ PyDoc_STRVAR(oct_doc,
 Return the octal representation of an integer or long integer.");
 
 
+static PyObject *
+builtin_open(PyObject *self, PyObject *args, PyObject *kwds)
+{
+       return PyObject_Call((PyObject*)&PyFile_Type, args, kwds);
+}
+
+PyDoc_STRVAR(open_doc,
+"open(name[, mode[, buffering]]) -> file object\n\
+\n\
+Open a file using the file() type, returns a file object.");
+
+
 static PyObject *
 builtin_ord(PyObject *self, PyObject* obj)
 {
@@ -2247,6 +2259,7 @@ static PyMethodDef builtin_methods[] = {
        {"max",         (PyCFunction)builtin_max,        METH_VARARGS | METH_KEYWORDS, max_doc},
        {"min",         (PyCFunction)builtin_min,        METH_VARARGS | METH_KEYWORDS, min_doc},
        {"oct",         builtin_oct,        METH_O, oct_doc},
+       {"open",        (PyCFunction)builtin_open,       METH_VARARGS | METH_KEYWORDS, open_doc},
        {"ord",         builtin_ord,        METH_O, ord_doc},
        {"pow",         builtin_pow,        METH_VARARGS, pow_doc},
        {"range",       builtin_range,      METH_VARARGS, range_doc},
@@ -2313,6 +2326,7 @@ _PyBuiltin_Init(void)
 #endif
        SETBUILTIN("dict",              &PyDict_Type);
        SETBUILTIN("enumerate",         &PyEnum_Type);
+       SETBUILTIN("file",              &PyFile_Type);
        SETBUILTIN("float",             &PyFloat_Type);
        SETBUILTIN("frozenset",         &PyFrozenSet_Type);
        SETBUILTIN("property",          &PyProperty_Type);
@@ -2329,10 +2343,6 @@ _PyBuiltin_Init(void)
        SETBUILTIN("tuple",             &PyTuple_Type);
        SETBUILTIN("type",              &PyType_Type);
        SETBUILTIN("xrange",            &PyRange_Type);
-
-       /* Note that open() is just an alias of file(). */
-       SETBUILTIN("open",              &PyFile_Type);
-       SETBUILTIN("file",              &PyFile_Type);
 #ifdef Py_USING_UNICODE
        SETBUILTIN("unicode",           &PyUnicode_Type);
 #endif