Поддержка Python 3 #5
Conversation
|
Привет! Я по строкам комментарии оставлю. |
| if isinstance(s, unicode): | ||
| s = s.encode(REQUEST_ENCODING) | ||
| except NameError: | ||
| pass |
There was a problem hiding this comment.
Под Python 3.x поведение оказывается неочевидным после этого изменения: функция не кодирует данные. Если она называется _encode, то и должна кодировать данные, хоть под 2.х, хоть под 3.х.
Кроме того, у json.dumps под 3.х нет метода encoding.
This reverts commit 482626c.
|
Пересмотрел замечания. |
|
Ага, так лучше. Я, наверное, замучал уже придирками, но мне все еще не до конца нравится код (особенно ветвление между 2.х и 3.х через отлов исключений), т.к. он непонятный. Например, неочевидно (и imho неправильно), что json_iterparse ожидает байтовые строки под 2.х и юникодные (?) под 3.х; неочевидно, какая из веток там выполняется под 2.х, а какая под 3.х. То же самое с функцией signature - типы локальнах переменных там отличаются под 2.х и 3.х; это, по сути, 2 разные функции под 2.х и 3.х. Из кода я не понял, почему там ловится UnicodeEncodeError (и в каких случаях это исключение может выпасть - при нормальной работе его быть ведь не должно). Вопрос можно решить добавлением комментариев в нужные места и небольшой переделкой кода - чтоб интерфейсы под 2.х и 3.х оставались одинаковыми (даже для приватного кода): чтоб было всегда четко определено, в каких переменных байты, а в каких юникод. |
Привет, попробовал портировать на PY3, вроде получилось, но опыта мало в этом деле, хотелось бы получить фидбэк если что-то не так.
Пришлось отказаться от поддержки PY25, т.к. под него нет requests, избавился от некоторых тестов которые на мой взгляд больше не нужны.
Тесты не смог прогнать только с PyPy, из-за локальных проблем с ним.