Martin v. Löwis [EMAIL PROTECTED] added the comment:
What's the purpose of the additional YY substring?
--
nosy: +loewis
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4004
___
STINNER Victor [EMAIL PROTECTED] added the comment:
@amaury.forgeotdarc: Done for out of memory.
@loewis: Oops, it's an error. I removed the YY.
So here is a new patch.
Added file: http://bugs.python.org/file11728/argv_error_newline-2.patch
___
Python
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment:
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
--
nosy: +amaury.forgeotdarc
Amaury Forgeot d'Arc [EMAIL PROTECTED] added the comment:
Committed as r66838.
--
resolution: accepted - fixed
status: open - closed
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4004
___
New submission from STINNER Victor [EMAIL PROTECTED]:
Example:
$ ./python $(echo -e \xff); ./python $(echo -e \xff); echo --
Could not convert argument 1 to stringCould not convert argument 1 to
string--
--
files: argv_error_newline.patch
keywords: patch
messages: 74102
nosy: haypo