[Python-3000-checkins] r59213 - in python/branches/py3k: Lib/test/test_complex.py Objects/complexobject.c
christian.heimes
python-3000-checkins at python.org
Wed Nov 28 11:04:30 CET 2007
Author: christian.heimes
Date: Wed Nov 28 11:04:30 2007
New Revision: 59213
Modified:
python/branches/py3k/ (props changed)
python/branches/py3k/Lib/test/test_complex.py
python/branches/py3k/Objects/complexobject.c
Log:
Merged revisions 59202-59211 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r59203 | guido.van.rossum | 2007-11-27 23:38:36 +0100 (Tue, 27 Nov 2007) | 4 lines
Patch # 1507 by Mark Dickinson. Make complex(x, -0) retain the sign of
the imaginary part (as long as it's not complex).
Backport candidate?
........
r59204 | christian.heimes | 2007-11-28 00:16:44 +0100 (Wed, 28 Nov 2007) | 2 lines
Expose Py_Py3kWarningFlag as sys.py3kwarning as discussed in #1504
Also added a warning.warnpy3k() as convenient method for Python 3.x related deprecation warnings.
........
r59206 | christian.heimes | 2007-11-28 00:53:14 +0100 (Wed, 28 Nov 2007) | 1 line
I forgot to fix one occurence of new in test_descr
........
r59208 | christian.heimes | 2007-11-28 09:02:36 +0100 (Wed, 28 Nov 2007) | 1 line
Added py3kwarning to the documentation of the sys module.
........
Modified: python/branches/py3k/Lib/test/test_complex.py
==============================================================================
--- python/branches/py3k/Lib/test/test_complex.py (original)
+++ python/branches/py3k/Lib/test/test_complex.py Wed Nov 28 11:04:30 2007
@@ -2,6 +2,7 @@
from test import test_support
from random import random
+from math import atan2
# These tests ensure that complex math does the right thing
@@ -207,6 +208,18 @@
self.assertAlmostEqual(complex(real=17+23j, imag=23), 17+46j)
self.assertAlmostEqual(complex(real=1+2j, imag=3+4j), -3+5j)
+ # check that the sign of a zero in the real or imaginary part
+ # is preserved when constructing from two floats. (These checks
+ # are harmless on systems without support for signed zeros.)
+ def split_zeros(x):
+ """Function that produces different results for 0. and -0."""
+ return atan2(x, -1.)
+
+ self.assertEqual(split_zeros(complex(1., 0.).imag), split_zeros(0.))
+ self.assertEqual(split_zeros(complex(1., -0.).imag), split_zeros(-0.))
+ self.assertEqual(split_zeros(complex(0., 1.).real), split_zeros(0.))
+ self.assertEqual(split_zeros(complex(-0., 1.).real), split_zeros(-0.))
+
c = 3.14 + 1j
self.assert_(complex(c) is c)
del c
Modified: python/branches/py3k/Objects/complexobject.c
==============================================================================
--- python/branches/py3k/Objects/complexobject.c (original)
+++ python/branches/py3k/Objects/complexobject.c Wed Nov 28 11:04:30 2007
@@ -809,6 +809,8 @@
PyNumberMethods *nbr, *nbi = NULL;
Py_complex cr, ci;
int own_r = 0;
+ int cr_is_complex = 0;
+ int ci_is_complex = 0;
static PyObject *complexstr;
static char *kwlist[] = {"real", "imag", 0};
@@ -889,6 +891,7 @@
retaining its real & imag parts here, and the return
value is (properly) of the builtin complex type. */
cr = ((PyComplexObject*)r)->cval;
+ cr_is_complex = 1;
if (own_r) {
Py_DECREF(r);
}
@@ -897,7 +900,6 @@
/* The "real" part really is entirely real, and contributes
nothing in the imaginary direction.
Just treat it as a double. */
- cr.imag = 0.0;
tmp = PyNumber_Float(r);
if (own_r) {
/* r was a newly created complex number, rather
@@ -917,15 +919,14 @@
}
if (i == NULL) {
ci.real = 0.0;
- ci.imag = 0.0;
}
- else if (PyComplex_Check(i))
+ else if (PyComplex_Check(i)) {
ci = ((PyComplexObject*)i)->cval;
- else {
+ ci_is_complex = 1;
+ } else {
/* The "imag" part really is entirely imaginary, and
contributes nothing in the real direction.
Just treat it as a double. */
- ci.imag = 0.0;
tmp = (*nbi->nb_float)(i);
if (tmp == NULL)
return NULL;
@@ -933,11 +934,16 @@
Py_DECREF(tmp);
}
/* If the input was in canonical form, then the "real" and "imag"
- parts are real numbers, so that ci.real and cr.imag are zero.
+ parts are real numbers, so that ci.imag and cr.imag are zero.
We need this correction in case they were not real numbers. */
- cr.real -= ci.imag;
- cr.imag += ci.real;
- return complex_subtype_from_c_complex(type, cr);
+
+ if (ci_is_complex) {
+ cr.real -= ci.imag;
+ }
+ if (cr_is_complex) {
+ ci.real += cr.imag;
+ }
+ return complex_subtype_from_doubles(type, cr.real, ci.real);
}
PyDoc_STRVAR(complex_doc,
More information about the Python-3000-checkins
mailing list