This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: missing newline in "Could not convert argument %s to string" error message
Type: behavior Stage:
Components: Versions: Python 3.0
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: amaury.forgeotdarc, loewis, vstinner
Priority: normal Keywords: patch

Created on 2008-09-30 22:38 by vstinner, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
argv_error_newline.patch vstinner, 2008-09-30 22:38 Add newline to the error message
argv_error_newline-2.patch vstinner, 2008-10-07 13:33 Add newline to the error messages
Messages (5)
msg74102 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2008-09-30 22:38
Example:

$ ./python $(echo -e "\xff"); ./python $(echo -e "\xff"); echo "--"
Could not convert argument 1 to stringCould not convert argument 1 to 
string--
msg74444 - (view) Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * (Python committer) Date: 2008-10-07 13:13
The patch is good, and consistent with all usages of fprintf(stderr) in
pythonrun.c, for example.

Please also add \n to
    fprintf(stderr, "out of memory");
in both python.c and frozenmain.c
msg74446 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2008-10-07 13:23
What's the purpose of the additional YY substring?
msg74447 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2008-10-07 13:33
@amaury.forgeotdarc: Done for "out of memory".

@loewis: Oops, it's an error. I removed the YY.

So here is a new patch.
msg74485 - (view) Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * (Python committer) Date: 2008-10-07 21:07
Committed as r66838.
History
Date User Action Args
2022-04-11 14:56:39adminsetgithub: 48254
2008-10-07 21:07:15amaury.forgeotdarcsetstatus: open -> closed
resolution: accepted -> fixed
messages: + msg74485
2008-10-07 13:33:08vstinnersetfiles: + argv_error_newline-2.patch
messages: + msg74447
2008-10-07 13:23:11loewissetnosy: + loewis
messages: + msg74446
2008-10-07 13:13:50amaury.forgeotdarcsetresolution: accepted
messages: + msg74444
nosy: + amaury.forgeotdarc
2008-09-30 22:38:35vstinnercreate