On Thu, Apr 24, 2014 at 02:24:39PM +0200, Stepan Kasal wrote: > From: Jean-Jacques Lafay <jeanjacques.lafay@xxxxxxxxx> > > In large repos, the recursion implementation of contains(commit, > commit_list) may result in a stack overflow. Replace the recursion with > a loop to fix it. > > This problem is more apparent on Windows than on Linux, where the stack > is more limited by default. > > See also this thread on the msysGit list: > > https://groups.google.com/d/topic/msysgit/FqT6boJrb2g/discussion > > [jes: re-written to imitate the original recursion more closely] > > Thomas Braun pointed out several documentation shortcomings. > > Tests are run only if ulimit -s is available. This means they cannot > be run on Windows. > > Signed-off-by: Jean-Jacques Lafay <jeanjacques.lafay@xxxxxxxxx> > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > Tested-by: Stepan Kasal <kasal@xxxxxx> Thanks, this version looks good to me. -Peff -- 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