Jonathan Nieder <jrnieder@xxxxxxxxx> writes: > Jeff King wrote: > >> What I missed is that copy_begin and copy_end here are actually size_t >> variables, not the pointers. Sorry for the confusion, and here's an >> updated version of the patch with this paragraph amended (the patch >> itself is identical): > > Subtle. The world makes more sense now. Thanks for figuring it out. > ... > For what it's worth, > Reviewed-by: Jonathan Nieder <jrnieder@xxxxxxxxx> Thanks, both. It seems that I missed all the fun ;-)