Message158366
> (1) The patch appears to assume that a Unicode string created with PyUnicode_New(size, 127) will have 'kind' PyUnicode_1BYTE_KIND. While this might be true in the current implementation, I don't know whether this is guaranteed in general. Martin, any comments on this?
The same assumption is used above in long_to_decimal_string(). I added
the same assert and changed maxchar to 'f'.
> (2) The patch doesn't compile with '--with-pydebug': there's a reference to the (now) undefined variable 'buffer' in one of the asserts.
Fixed.
> (3) The overflow check looks as though it needs to be reworked.
I was left an old constraint (it is enough), but it really can be
weakened (in fact, it can be so weakened in the current code).
Thank you for the comments and the found error. |
|
Date |
User |
Action |
Args |
2012-04-15 20:32:12 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, loewis, mark.dickinson, pitrou, eric.smith, ezio.melotti, skrah |
2012-04-15 20:32:11 | serhiy.storchaka | link | issue14339 messages |
2012-04-15 20:32:11 | serhiy.storchaka | create | |
|