When the second argument of complex() is not a number/string, the type error reports the error but prints the type of the first argument: > complex({1:2},1j) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() argument must be a string or a number, not 'dict' >complex(1j,{1:2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() argument must be a string or a number, not 'complex' >>> complex(1, {1:2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() argument must be a string or a number, not 'int'

Contains changes made to Objects/complexobject.c Changed the error message returned when second argument of complex() is not number/string Originally: >complex(1j,{1:2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() argument must be a string or a number, not 'complex' After patch: >complex(1j,{1:2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() argument must be a string or a number, not 'dict'

Please upload your patch from a Mercurial clone: * https://docs.python.org/devguide/setup.html#checkout * https://docs.python.org/devguide/patch.html Currently, if you pass a string as a second argument, you get: >>> complex(1, "1") Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() second arg can't be a string So the exception message should probably be changed to include "second arg" or "second argument": >>> complex(1j, {1: 2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() second arg must be a number, not 'dict' Also, there is already a check for second argument in line 952 so the "must be a string" part is probably not needed. We probably need to check whether these two cases can be combined. You also need to add some tests to Lib/test/test_complex.py.

@manishearth: Nice catch! Thanks for the report. @soummyaah: Thanks for the patch. Please could you sign a contributor agreement[1], so that we can commit the patch? As Berker says, tests would be good to have, too. [1] https://www.python.org/psf/contrib/contrib-form/

This should be fixed in Python 3.5, too.

I've signed the contributor agreement form. I think it said that it'll take a few days to process? I've made the changes requested and am currently working on the tests. Will submit a new patch file containing all required changes soon.

Thank you! I look forward to the new patch.

Changed error message to: >>> complex({1:2},1) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() first arg must be a string or a number, not 'dict' >>> complex(1j, {1: 2}) Traceback (most recent call last): File "<stdin>", line 1, in <module> TypeError: complex() second arg must be a number, not 'dict' Added tests to check the error raised.

Apologies. This is the correct file.

Squashed the commits for better readability. Also, change required in Python 3.4 as well

Thanks for the patch. We can't fix this in 3.4 because it's in security-fix-only mode: https://docs.python.org/devguide/index.html#status-of-python-branches

Thanks. I'll apply this shortly (but probably not before the weekend).

New changeset 92f4ce2d5ebb by Mark Dickinson in branch '3.5': Issue #28203: Fix incorrect type in error message from complex(1.0, {2:3}). Patch by Soumya Sharma. https://hg.python.org/cpython/rev/92f4ce2d5ebb New changeset a2d93e6bcbcf by Mark Dickinson in branch '3.6': Issue #28203: Merge from 3.5 https://hg.python.org/cpython/rev/a2d93e6bcbcf New changeset 9790bc211107 by Mark Dickinson in branch 'default': Issue #28203: Merge from 3.6 https://hg.python.org/cpython/rev/9790bc211107

Fixed; thanks. I made a couple of changes: - Use "argument" rather than "arg", to be consistent with the original code (but admittedly not consistent with the rest of the module, where there doesn't seem to be any consistent choice between "arg" and "argument"). - Reformat C and test code to avoid long lines. - Slight rearrangement of the C code so that all of the "i" handling is in one place.

Thanks for the merge!