Author amaury.forgeotdarc
Recipients amaury.forgeotdarc, barry, dlitz, jmfauth, ocean-city, pitrou, vstinner
Date 2008-10-07.10:49:47
SpamBayes Score 1.7947e-09
Marked as misclassified No
Message-id <>
Your patch does the correct thing, however an explanation of the -1
value would be welcome. Something like:
/* The file has been reopened; parsing will restart from 
 * the beginning of the file, we have to reset the line number.
 * But this function has been called from inside tok_nextc() which 
 * will increment lineno before it returns. So we set it -1 so that
 * the next call to tok_nextc() will start with tok->lineno == 0.

Or we could change the place of the tok->lineno++ in tok_nextc() so that
it is called before the call to decoding_fgets(); other changes will be

Then, I think that your test is not correct: What is the meaning of the
following line?
(the module "traceback" has no attribute "tp_lineno")
I presume that you intended something like:
and test at some point that "process.returncode == lineno"
Date User Action Args
2008-10-07 10:49:49amaury.forgeotdarcsetrecipients: + amaury.forgeotdarc, barry, pitrou, vstinner, ocean-city, jmfauth, dlitz
2008-10-07 10:49:48amaury.forgeotdarcsetmessageid: <>
2008-10-07 10:49:47amaury.forgeotdarclinkissue2384 messages
2008-10-07 10:49:47amaury.forgeotdarccreate