2009/1/27 Kjetil Barvik <barvik@xxxxxxxxxxxx> > Mike Ralphson <mike.ralphson@xxxxxxxxx> writes: > > > 2009/1/26 Kjetil Barvik <barvik@xxxxxxxxxxxx>: > >> Currently inside show_patch_diff() we have and fstat() call after an > >> ok lstat() call. Since we before the call to fstat() have already > >> test for the link case with S_ISLNK() the fstat() can be removed. > > > > s/have and/have an/ ? > > Correct! Will fix. Thanks! Typo stood out and blinded me to the rest of the grammar... How about: Currently inside show_patch_diff() we have an fstat() call after an ok lstat() call. Since before the call to fstat() we have already tested for the link case with S_ISLNK(), the fstat() can be removed. 'Half-a-job' Mike -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html