Message147362
Looking at it again, the intention was probably to increment
collend so that it points to the first non-garbage character
(or '\0'). If that's the case, the loop should be something
like this:
while (collend < end) {
if ((0 < *collend && *collend < 256) ||
Py_UNICODE_ISSPACE(*collend) ||
Py_UNICODE_TODECIMAL(*collend) >= 0)
break;
collend++;
}
I can understand if no one wants to touch this given that
the function is deprecated, provided that you agree that
the existing code is redundant but harmless; i.e. it cannot
enter an infinite loop. |
|
Date |
User |
Action |
Args |
2011-11-09 18:31:24 | skrah | set | recipients:
+ skrah, loewis, rhettinger, facundobatista, mark.dickinson, vstinner |
2011-11-09 18:31:24 | skrah | set | messageid: <1320863484.43.0.72304947973.issue13093@psf.upfronthosting.co.za> |
2011-11-09 18:31:23 | skrah | link | issue13093 messages |
2011-11-09 18:31:23 | skrah | create | |
|