Message215293
Thanks for the feedback! I have attached an updated patch.
I did not include any testcase because the patch did not create any new code paths, so I was assuming it'd be covered by the existing test case. But of course I was wrong. In the revised patch, I added a testcase based on your example of a more complex memoryview.
(Note, however, that even with the previous implementation using nbytes = PySequence_Size(input_chunk) this test does not fail, because nbytes is used only to estimate the size of the text string). |
|
Date |
User |
Action |
Args |
2014-04-01 04:19:11 | nikratio | set | recipients:
+ nikratio, pitrou, vstinner, serhiy.storchaka |
2014-04-01 04:19:11 | nikratio | set | messageid: <1396325951.32.0.124139014868.issue21057@psf.upfronthosting.co.za> |
2014-04-01 04:19:11 | nikratio | link | issue21057 messages |
2014-04-01 04:19:10 | nikratio | create | |
|