Thank you for the fix to some problems I didn't know we had. Thank you also for the explanatory docstring and updating the rest of the code to use the fix.
I have no further reservations about merging this.
+1
@Vincent: Should we target 7.1 and then merge up to trunk?
Thank you for the fix to some problems I didn't know we had. Thank you also for the explanatory docstring and updating the rest of the code to use the fix.
I have no further reservations about merging this.
+1
@Vincent: Should we target 7.1 and then merge up to trunk?