Message147966
Hi!
I'd like to have this committed to be able to fix #13349. So here's a review.
- In Doc/c-api/unicode.rst, the two "versionchanged:: 3.3" directives can be merged
- In tests, I'd use 'abcde' rather than 'xxxxx' to make sure that correct characters are copied to the output (hope you understand what I mean)
- No test checks that width and precision work on characters rather than bytes
- The changes to unicodeobject.c don't apply on top of current default branch. |
|
Date |
User |
Action |
Args |
2011-11-19 19:30:37 | petri.lehtinen | set | recipients:
+ petri.lehtinen, lemburg, mark.dickinson, vstinner, eric.smith, ron_adam, ezio.melotti, lekma, ysj.ray, python-dev |
2011-11-19 19:30:37 | petri.lehtinen | set | messageid: <1321731037.11.0.804993812878.issue7330@psf.upfronthosting.co.za> |
2011-11-19 19:30:36 | petri.lehtinen | link | issue7330 messages |
2011-11-19 19:30:36 | petri.lehtinen | create | |
|