This repository was archived by the owner on Nov 9, 2017. It is now read-only.
File tree Expand file tree Collapse file tree 2 files changed +5
-7
lines changed Expand file tree Collapse file tree 2 files changed +5
-7
lines changed Original file line number Diff line number Diff line change @@ -99,12 +99,11 @@ static int contains_test(struct commit *candidate,
99
99
}
100
100
101
101
/*
102
- * This stack on the heap mimics the real stack in the previous implementation
103
- * of contains_recurse() that had to be replaced because it easily led to a
104
- * stack overflow.
102
+ * Mimicking the real stack, this stack lives on the heap, avoiding stack
103
+ * overflows.
105
104
*
106
- * Every stack item points at a certain commit whose parents are iterated over.
107
- * Each item's current parent is the current next stack item's commit .
105
+ * At each recursion step, the stack items points to the commits whose
106
+ * ancestors are to be inspected .
108
107
*/
109
108
struct stack {
110
109
int nr , alloc ;
Original file line number Diff line number Diff line change @@ -1380,9 +1380,8 @@ test_expect_success 'multiple --points-at are OR-ed together' '
1380
1380
test_cmp expect actual
1381
1381
'
1382
1382
1383
- # what about a deep repo ?
1384
-
1385
1383
>expect
1384
+ # ulimit is a bash builtin; we can rely on that in MinGW, but nowhere else
1386
1385
test_expect_success MINGW ' --contains works in a deep repo' '
1387
1386
ulimit -s 64
1388
1387
i=1 &&
You can’t perform that action at this time.
0 commit comments