Skip to content

Commit

Permalink
Remove the exceptions builtin module, all the exceptions are already …
Browse files Browse the repository at this point in the history
…builtin.
  • Loading branch information
nnorwitz committed Feb 26, 2007
1 parent f543348 commit 2633c69
Show file tree
Hide file tree
Showing 15 changed files with 34 additions and 66 deletions.
8 changes: 4 additions & 4 deletions Lib/ctypes/test/test_structures.py
Expand Up @@ -294,20 +294,20 @@ class Person(Structure):
# In Python 2.5, Exception is a new-style class, and the repr changed
if issubclass(Exception, object):
self.failUnlessEqual(msg,
"(Phone) <type 'exceptions.TypeError'>: "
"(Phone) <type 'TypeError'>: "
"expected string or Unicode object, int found")
else:
self.failUnlessEqual(msg,
"(Phone) exceptions.TypeError: "
"(Phone) TypeError: "
"expected string or Unicode object, int found")

cls, msg = self.get_except(Person, "Someone", ("a", "b", "c"))
self.failUnlessEqual(cls, RuntimeError)
if issubclass(Exception, object):
self.failUnlessEqual(msg,
"(Phone) <type 'exceptions.ValueError'>: too many initializers")
"(Phone) <type 'ValueError'>: too many initializers")
else:
self.failUnlessEqual(msg, "(Phone) exceptions.ValueError: too many initializers")
self.failUnlessEqual(msg, "(Phone) ValueError: too many initializers")


def get_except(self, func, *args):
Expand Down
6 changes: 3 additions & 3 deletions Lib/test/output/test_logging
Expand Up @@ -488,12 +488,12 @@ INFO:a.b.c.d:Info 5
-- log_test4 begin ---------------------------------------------------
config0: ok.
config1: ok.
config2: <type 'exceptions.AttributeError'>
config3: <type 'exceptions.KeyError'>
config2: <type 'AttributeError'>
config3: <type 'KeyError'>
-- log_test4 end ---------------------------------------------------
-- log_test5 begin ---------------------------------------------------
ERROR:root:just testing
<type 'exceptions.KeyError'>... Don't panic!
<type 'KeyError'>... Don't panic!
-- log_test5 end ---------------------------------------------------
-- logrecv output begin ---------------------------------------------------
ERR -> CRITICAL: Message 0 (via logrecv.tcp.ERR)
Expand Down
11 changes: 1 addition & 10 deletions Lib/test/test_exceptions.py
Expand Up @@ -15,15 +15,6 @@

class ExceptionTests(unittest.TestCase):

def testReload(self):
# Reloading the built-in exceptions module failed prior to Py2.2, while it
# should act the same as reloading built-in sys.
try:
import exceptions
reload(exceptions)
except ImportError as e:
self.fail("reloading exceptions: %s" % e)

def raise_catch(self, exc, excname):
try:
raise exc, "spam"
Expand Down Expand Up @@ -289,7 +280,7 @@ def testAttributes(self):
if type(e) is not exc:
raise
# Verify module name
self.assertEquals(type(e).__module__, 'exceptions')
self.assertEquals(type(e).__module__, '__builtin__')
# Verify no ref leaks in Exc_str()
s = str(e)
for checkArgName in expected:
Expand Down
4 changes: 2 additions & 2 deletions Lib/test/test_generators.py
Expand Up @@ -1691,7 +1691,7 @@ def printsolution(self, x):
>>> g.next()
>>> del g
>>> sys.stderr.getvalue().startswith(
... "Exception exceptions.RuntimeError: 'generator ignored GeneratorExit' in "
... "Exception RuntimeError: 'generator ignored GeneratorExit' in "
... )
True
>>> sys.stderr = old
Expand Down Expand Up @@ -1808,7 +1808,7 @@ def printsolution(self, x):
... del l
... err = sys.stderr.getvalue().strip()
... err.startswith(
... "Exception exceptions.RuntimeError: RuntimeError() in <"
... "Exception RuntimeError: RuntimeError() in <"
... )
... err.endswith("> ignored")
... len(err.splitlines())
Expand Down
14 changes: 7 additions & 7 deletions Lib/test/test_modulefinder.py
Expand Up @@ -87,7 +87,7 @@
"a.module",
["a", "a.module",
"b", "b.x", "b.y", "b.z",
"__future__", "sys", "exceptions"],
"__future__", "sys", "gc"],
["blahblah", "z"], [],
"""\
mymodule.py
Expand All @@ -96,11 +96,11 @@
from __future__ import absolute_import
import sys # sys
import blahblah # fails
import exceptions # exceptions
import gc # gc
import b.x # b.x
from b import y # b.y
from b.z import * # b.z.*
a/exceptions.py
a/gc.py
a/sys.py
import mymodule
a/b/__init__.py
Expand All @@ -123,16 +123,16 @@
"a.b.c", "a.b.c.moduleC",
"a.b.c.d", "a.b.c.e",
"a.b.x",
"exceptions"],
"gc"],
[], [],
"""\
mymodule.py
a/__init__.py
from .b import y, z # a.b.y, a.b.z
a/module.py
from __future__ import absolute_import # __future__
import exceptions # exceptions
a/exceptions.py
import gc # gc
a/gc.py
a/sys.py
a/b/__init__.py
from ..b import x # a.b.x
Expand Down Expand Up @@ -170,7 +170,7 @@
a/another.py
a/module.py
from .b import y, z # a.b.y, a.b.z
a/exceptions.py
a/gc.py
a/sys.py
a/b/__init__.py
from .c import moduleC # a.b.c.moduleC
Expand Down
6 changes: 3 additions & 3 deletions Lib/test/test_weakref.py
Expand Up @@ -104,9 +104,9 @@ def test_proxy_ref(self):
def check(proxy):
proxy.bar

self.assertRaises(weakref.ReferenceError, check, ref1)
self.assertRaises(weakref.ReferenceError, check, ref2)
self.assertRaises(weakref.ReferenceError, bool, weakref.proxy(C()))
self.assertRaises(ReferenceError, check, ref1)
self.assertRaises(ReferenceError, check, ref2)
self.assertRaises(ReferenceError, bool, weakref.proxy(C()))
self.assert_(self.cbcalled == 2)

def check_basic_ref(self, factory):
Expand Down
2 changes: 1 addition & 1 deletion Lib/traceback.py
Expand Up @@ -169,7 +169,7 @@ def format_exception_only(etype, value):

stype = etype.__name__
smod = etype.__module__
if smod not in ("exceptions", "__main__", "__builtin__"):
if smod not in ("__main__", "__builtin__"):
stype = smod + '.' + stype

if not issubclass(etype, SyntaxError):
Expand Down
2 changes: 0 additions & 2 deletions Lib/weakref.py
Expand Up @@ -20,8 +20,6 @@
ProxyType,
ReferenceType)

from exceptions import ReferenceError


ProxyTypes = (ProxyType, CallableProxyType)

Expand Down
2 changes: 2 additions & 0 deletions Misc/NEWS
Expand Up @@ -28,6 +28,8 @@ TO DO
Core and Builtins
-----------------

- Remove the exceptions module, all the exceptions are already builtin.

- input() becomes raw_input(): the name input() now implements the
functionality formerly known as raw_input(); the name raw_input()
is no longer defined.
Expand Down
36 changes: 8 additions & 28 deletions Objects/exceptions.c
Expand Up @@ -10,20 +10,11 @@
#include "osdefs.h"

#define MAKE_IT_NONE(x) (x) = Py_None; Py_INCREF(Py_None);
#define EXC_MODULE_NAME "exceptions."

/* NOTE: If the exception class hierarchy changes, don't forget to update
* Lib/test/exception_hierarchy.txt
*/

PyDoc_STRVAR(exceptions_doc, "Python's standard exception class hierarchy.\n\
\n\
Exceptions found here are defined both in the exceptions module and the\n\
built-in namespace. It is recommended that user-defined exceptions\n\
inherit from Exception. See the documentation for the exception\n\
inheritance hierarchy.\n\
");

/*
* BaseException
*/
Expand Down Expand Up @@ -282,7 +273,7 @@ static PyGetSetDef BaseException_getset[] = {
static PyTypeObject _PyExc_BaseException = {
PyObject_HEAD_INIT(NULL)
0, /*ob_size*/
EXC_MODULE_NAME "BaseException", /*tp_name*/
"BaseException", /*tp_name*/
sizeof(PyBaseExceptionObject), /*tp_basicsize*/
0, /*tp_itemsize*/
(destructor)BaseException_dealloc, /*tp_dealloc*/
Expand Down Expand Up @@ -333,7 +324,7 @@ PyObject *PyExc_BaseException = (PyObject *)&_PyExc_BaseException;
static PyTypeObject _PyExc_ ## EXCNAME = { \
PyObject_HEAD_INIT(NULL) \
0, \
EXC_MODULE_NAME # EXCNAME, \
# EXCNAME, \
sizeof(PyBaseExceptionObject), \
0, (destructor)BaseException_dealloc, 0, 0, 0, 0, 0, 0, 0, \
0, 0, 0, 0, 0, 0, 0, \
Expand All @@ -349,7 +340,7 @@ PyObject *PyExc_ ## EXCNAME = (PyObject *)&_PyExc_ ## EXCNAME
static PyTypeObject _PyExc_ ## EXCNAME = { \
PyObject_HEAD_INIT(NULL) \
0, \
EXC_MODULE_NAME # EXCNAME, \
# EXCNAME, \
sizeof(Py ## EXCSTORE ## Object), \
0, (destructor)EXCSTORE ## _dealloc, 0, 0, 0, 0, 0, 0, 0, 0, 0, \
0, 0, 0, 0, 0, \
Expand All @@ -365,7 +356,7 @@ PyObject *PyExc_ ## EXCNAME = (PyObject *)&_PyExc_ ## EXCNAME
static PyTypeObject _PyExc_ ## EXCNAME = { \
PyObject_HEAD_INIT(NULL) \
0, \
EXC_MODULE_NAME # EXCNAME, \
# EXCNAME, \
sizeof(Py ## EXCSTORE ## Object), 0, \
(destructor)EXCSTORE ## _dealloc, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, \
(reprfunc)EXCSTR, 0, 0, 0, \
Expand Down Expand Up @@ -1632,7 +1623,7 @@ UnicodeEncodeError_str(PyObject *self)
static PyTypeObject _PyExc_UnicodeEncodeError = {
PyObject_HEAD_INIT(NULL)
0,
EXC_MODULE_NAME "UnicodeEncodeError",
"UnicodeEncodeError",
sizeof(PyUnicodeErrorObject), 0,
(destructor)UnicodeError_dealloc, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
(reprfunc)UnicodeEncodeError_str, 0, 0, 0,
Expand Down Expand Up @@ -1704,7 +1695,7 @@ UnicodeDecodeError_str(PyObject *self)
static PyTypeObject _PyExc_UnicodeDecodeError = {
PyObject_HEAD_INIT(NULL)
0,
EXC_MODULE_NAME "UnicodeDecodeError",
"UnicodeDecodeError",
sizeof(PyUnicodeErrorObject), 0,
(destructor)UnicodeError_dealloc, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
(reprfunc)UnicodeDecodeError_str, 0, 0, 0,
Expand Down Expand Up @@ -1802,7 +1793,7 @@ UnicodeTranslateError_str(PyObject *self)
static PyTypeObject _PyExc_UnicodeTranslateError = {
PyObject_HEAD_INIT(NULL)
0,
EXC_MODULE_NAME "UnicodeTranslateError",
"UnicodeTranslateError",
sizeof(PyUnicodeErrorObject), 0,
(destructor)UnicodeError_dealloc, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
(reprfunc)UnicodeTranslateError_str, 0, 0, 0,
Expand Down Expand Up @@ -1956,17 +1947,10 @@ SimpleExtendsException(PyExc_Warning, UnicodeWarning,
*/
PyObject *PyExc_MemoryErrorInst=NULL;

/* module global functions */
static PyMethodDef functions[] = {
/* Sentinel */
{NULL, NULL}
};

#define PRE_INIT(TYPE) if (PyType_Ready(&_PyExc_ ## TYPE) < 0) \
Py_FatalError("exceptions bootstrapping error.");

#define POST_INIT(TYPE) Py_INCREF(PyExc_ ## TYPE); \
PyModule_AddObject(m, # TYPE, PyExc_ ## TYPE); \
if (PyDict_SetItemString(bdict, # TYPE, PyExc_ ## TYPE)) \
Py_FatalError("Module dictionary insertion problem.");

Expand Down Expand Up @@ -1996,7 +1980,7 @@ InvalidParameterHandler(
PyMODINIT_FUNC
_PyExc_Init(void)
{
PyObject *m, *bltinmod, *bdict;
PyObject *bltinmod, *bdict;

PRE_INIT(BaseException)
PRE_INIT(Exception)
Expand Down Expand Up @@ -2053,10 +2037,6 @@ _PyExc_Init(void)
PRE_INIT(ImportWarning)
PRE_INIT(UnicodeWarning)

m = Py_InitModule4("exceptions", functions, exceptions_doc,
(PyObject *)NULL, PYTHON_API_VERSION);
if (m == NULL) return;

bltinmod = PyImport_ImportModule("__builtin__");
if (bltinmod == NULL)
Py_FatalError("exceptions bootstrapping error.");
Expand Down
1 change: 0 additions & 1 deletion PC/config.c
Expand Up @@ -161,7 +161,6 @@ struct _inittab _PyImport_Inittab[] = {
{"__main__", NULL},
{"__builtin__", NULL},
{"sys", NULL},
{"exceptions", NULL},

{"_types", init_types},

Expand Down
1 change: 0 additions & 1 deletion PC/os2emx/config.c
Expand Up @@ -172,7 +172,6 @@ struct _inittab _PyImport_Inittab[] = {
{"__main__", NULL},
{"__builtin__", NULL},
{"sys", NULL},
{"exceptions", NULL},

/* This lives in gcmodule.c */
{"gc", initgc},
Expand Down
3 changes: 2 additions & 1 deletion Python/errors.c
Expand Up @@ -612,7 +612,8 @@ PyErr_WriteUnraisable(PyObject *obj)
PyFile_WriteString("<unknown>", f);
else {
char* modstr = PyString_AsString(moduleName);
if (modstr)
if (modstr &&
strcmp(modstr, "__builtin__") != 0)
{
PyFile_WriteString(modstr, f);
PyFile_WriteString(".", f);
Expand Down
3 changes: 1 addition & 2 deletions Python/pythonrun.c
Expand Up @@ -221,7 +221,6 @@ Py_InitializeEx(int install_sigs)

/* initialize builtin exceptions */
_PyExc_Init();
_PyImport_FixupExtension("exceptions", "exceptions");

/* phase 2 of builtins */
_PyImport_FixupExtension("__builtin__", "__builtin__");
Expand Down Expand Up @@ -1168,7 +1167,7 @@ PyErr_Display(PyObject *exception, PyObject *value, PyObject *tb)
err = PyFile_WriteString("<unknown>", f);
else {
char* modstr = PyString_AsString(moduleName);
if (modstr && strcmp(modstr, "exceptions"))
if (modstr && strcmp(modstr, "__builtin__"))
{
err = PyFile_WriteString(modstr, f);
err += PyFile_WriteString(".", f);
Expand Down
1 change: 0 additions & 1 deletion RISCOS/Modules/config.c
Expand Up @@ -63,7 +63,6 @@ struct _inittab _PyImport_Inittab[] = {
{"__main__", NULL},
{"__builtin__", NULL},
{"sys", NULL},
{"exceptions", NULL},

/* This lives in gcmodule.c */
{"gc", initgc},
Expand Down

0 comments on commit 2633c69

Please sign in to comment.