Message216283
Banjamin, Your patch looks good to me!
I have a small concern regarding "We now know we will succeed..." -- should there be a test case to make sure fstat test here matches whatever test is/was done on a lower level?
Or is your code now such that it will explicitly succeed because it doesn't call anything that can fail after the comment? |
|
Date |
User |
Action |
Args |
2014-04-15 09:34:17 | Dima.Tisnek | set | recipients:
+ Dima.Tisnek, benjamin.peterson, python-dev |
2014-04-15 09:34:17 | Dima.Tisnek | set | messageid: <1397554457.58.0.99289799944.issue21191@psf.upfronthosting.co.za> |
2014-04-15 09:34:17 | Dima.Tisnek | link | issue21191 messages |
2014-04-15 09:34:17 | Dima.Tisnek | create | |
|