Kaydet (Commit) ccadf84a authored tarafından Georg Brandl's avatar Georg Brandl

Patch #1460496: round() now accepts keyword arguments.

üst 338ef7d2
...@@ -1395,6 +1395,9 @@ class BuiltinTest(unittest.TestCase): ...@@ -1395,6 +1395,9 @@ class BuiltinTest(unittest.TestCase):
self.assertEqual(round(-8.0, -1), -10.0) self.assertEqual(round(-8.0, -1), -10.0)
# test new kwargs
self.assertEqual(round(number=-8.0, ndigits=-1), -10.0)
self.assertRaises(TypeError, round) self.assertRaises(TypeError, round)
def test_setattr(self): def test_setattr(self):
......
...@@ -12,6 +12,8 @@ What's New in Python 2.5 alpha 1? ...@@ -12,6 +12,8 @@ What's New in Python 2.5 alpha 1?
Core and builtins Core and builtins
----------------- -----------------
- Patch #1460496: round() now accepts keyword arguments.
- Fixed bug #1459029 - unicode reprs were double-escaped. - Fixed bug #1459029 - unicode reprs were double-escaped.
- Patch #1396919: The system scope threads are reenabled on FreeBSD - Patch #1396919: The system scope threads are reenabled on FreeBSD
......
...@@ -1870,32 +1870,34 @@ For most object types, eval(repr(object)) == object."); ...@@ -1870,32 +1870,34 @@ For most object types, eval(repr(object)) == object.");
static PyObject * static PyObject *
builtin_round(PyObject *self, PyObject *args) builtin_round(PyObject *self, PyObject *args, PyObject *kwds)
{ {
double x; double number;
double f; double f;
int ndigits = 0; int ndigits = 0;
int i; int i;
static char *kwlist[] = {"number", "ndigits", 0};
if (!PyArg_ParseTuple(args, "d|i:round", &x, &ndigits)) if (!PyArg_ParseTupleAndKeywords(args, kwds, "d|i:round",
return NULL; kwlist, &number, &ndigits))
return NULL;
f = 1.0; f = 1.0;
i = abs(ndigits); i = abs(ndigits);
while (--i >= 0) while (--i >= 0)
f = f*10.0; f = f*10.0;
if (ndigits < 0) if (ndigits < 0)
x /= f; number /= f;
else else
x *= f; number *= f;
if (x >= 0.0) if (number >= 0.0)
x = floor(x + 0.5); number = floor(number + 0.5);
else else
x = ceil(x - 0.5); number = ceil(number - 0.5);
if (ndigits < 0) if (ndigits < 0)
x *= f; number *= f;
else else
x /= f; number /= f;
return PyFloat_FromDouble(x); return PyFloat_FromDouble(number);
} }
PyDoc_STRVAR(round_doc, PyDoc_STRVAR(round_doc,
...@@ -2248,7 +2250,7 @@ static PyMethodDef builtin_methods[] = { ...@@ -2248,7 +2250,7 @@ static PyMethodDef builtin_methods[] = {
{"reduce", builtin_reduce, METH_VARARGS, reduce_doc}, {"reduce", builtin_reduce, METH_VARARGS, reduce_doc},
{"reload", builtin_reload, METH_O, reload_doc}, {"reload", builtin_reload, METH_O, reload_doc},
{"repr", builtin_repr, METH_O, repr_doc}, {"repr", builtin_repr, METH_O, repr_doc},
{"round", builtin_round, METH_VARARGS, round_doc}, {"round", (PyCFunction)builtin_round, METH_VARARGS | METH_KEYWORDS, round_doc},
{"setattr", builtin_setattr, METH_VARARGS, setattr_doc}, {"setattr", builtin_setattr, METH_VARARGS, setattr_doc},
{"sorted", (PyCFunction)builtin_sorted, METH_VARARGS | METH_KEYWORDS, sorted_doc}, {"sorted", (PyCFunction)builtin_sorted, METH_VARARGS | METH_KEYWORDS, sorted_doc},
{"sum", builtin_sum, METH_VARARGS, sum_doc}, {"sum", builtin_sum, METH_VARARGS, sum_doc},
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment