[PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-04 Thread Erik Faye-Lund
in ab1a11be ("mingw_rmdir: set errno=ENOTEMPTY when appropriate"), a check was added to prevent us from retrying to delete a directory that is both in use and non-empty. However, this logic was slightly flawed; since we didn't return immediately, we end up falling out of the retry-loop, but right

Re: [PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-04 Thread Johannes Schindelin
Hi kusma, On Tue, 4 Dec 2012, Erik Faye-Lund wrote: > in ab1a11be ("mingw_rmdir: set errno=ENOTEMPTY when appropriate"), > a check was added to prevent us from retrying to delete a directory > that is both in use and non-empty. > > However, this logic was slightly flawed; since we didn't return

Re: [PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-05 Thread Erik Faye-Lund
Sorry for a late reply. On Tue, Dec 4, 2012 at 5:35 PM, Johannes Schindelin wrote: > Hi kusma, > > On Tue, 4 Dec 2012, Erik Faye-Lund wrote: > >> in ab1a11be ("mingw_rmdir: set errno=ENOTEMPTY when appropriate"), >> a check was added to prevent us from retrying to delete a directory >> that is bo

Re: [PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-05 Thread Johannes Schindelin
Hi kusma, On Wed, 5 Dec 2012, Erik Faye-Lund wrote: > Sorry for a late reply. Yeah, sorry, my replies tend to be delayed a lot. For the record: your reply was not at all late. > On Tue, Dec 4, 2012 at 5:35 PM, Johannes Schindelin > wrote: > > > > On Tue, 4 Dec 2012, Erik Faye-Lund wrote: > > >

Re: [PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-05 Thread Erik Faye-Lund
On Wed, Dec 5, 2012 at 5:02 PM, Johannes Schindelin wrote: > Hi kusma, > > On Wed, 5 Dec 2012, Erik Faye-Lund wrote: > >> Sorry for a late reply. > > Yeah, sorry, my replies tend to be delayed a lot. For the record: your > reply was not at all late. > >> On Tue, Dec 4, 2012 at 5:35 PM, Johannes Sc

Re: [PATCH] mingw_rmdir: do not prompt for retry when non-empty

2012-12-05 Thread Junio C Hamano
Erik Faye-Lund writes: > On Wed, Dec 5, 2012 at 5:02 PM, Johannes Schindelin > wrote: > ... > Since we're justifying the approaches, I'd like to explain why I > preferred the return approach: it performs less tests. While this > might sound like premature optimizations, performance is not why I