diff options
author | Fred Drake <fdrake@acm.org> | 2000-06-01 02:02:46 (GMT) |
---|---|---|
committer | Fred Drake <fdrake@acm.org> | 2000-06-01 02:02:46 (GMT) |
commit | 137507ea0308fcdab66c1eee700048c41ed7a7d7 (patch) | |
tree | d0be03f809710a00ebfe02b7d332531fda764c99 /Modules/arraymodule.c | |
parent | b5fc749c8b809f8ebab74cebff4153880236ca33 (diff) | |
download | cpython-137507ea0308fcdab66c1eee700048c41ed7a7d7.zip cpython-137507ea0308fcdab66c1eee700048c41ed7a7d7.tar.gz cpython-137507ea0308fcdab66c1eee700048c41ed7a7d7.tar.bz2 |
Michael Hudson <mwh21@cam.ac.uk>:
Removed PyErr_BadArgument() calls and replaced them with more useful
error messages.
Diffstat (limited to 'Modules/arraymodule.c')
-rw-r--r-- | Modules/arraymodule.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/Modules/arraymodule.c b/Modules/arraymodule.c index 9a09c7d..7966e0a 100644 --- a/Modules/arraymodule.c +++ b/Modules/arraymodule.c @@ -540,7 +540,9 @@ array_concat(a, bb) int size; arrayobject *np; if (!is_arrayobject(bb)) { - PyErr_BadArgument(); + PyErr_Format(PyExc_TypeError, + "can only append array (not \"%.200s\") to array", + bb->ob_type->tp_name); return NULL; } #define b ((arrayobject *)bb) @@ -613,7 +615,9 @@ array_ass_slice(a, ilow, ihigh, v) } } else { - PyErr_BadArgument(); + PyErr_Format(PyExc_TypeError, + "can only assign array (not \"%.200s\") to array slice", + v->ob_type->tp_name); return -1; } if (ilow < 0) @@ -821,7 +825,8 @@ array_reverse(self, args) char tmp[sizeof(double)]; /* Assume that's the max item size */ if (args != NULL) { - PyErr_BadArgument(); + PyErr_SetString(PyExc_TypeError, + "<array>.reverse requires exactly 0 arguments"); return NULL; } |