diffgrep_consume() already stops regexec() after hitting, so the value of this series is negligible. Tay Ray Chuan seems to touch xdiff code too and my changes are across many files. I'll drop this series. 2011/8/15 Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx>: > > Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> > --- > diffcore-pickaxe.c | 6 +----- > 1 files changed, 1 insertions(+), 5 deletions(-) > > diff --git a/diffcore-pickaxe.c b/diffcore-pickaxe.c > index 12811b9..e028dd5 100644 > --- a/diffcore-pickaxe.c > +++ b/diffcore-pickaxe.c > @@ -21,11 +21,7 @@ static int diffgrep_consume(void *priv, char *line, unsigned long len) > if (line[0] != '+' && line[0] != '-') > return 0; > if (data->hit) > - /* > - * NEEDSWORK: we should have a way to terminate the > - * caller early. > - */ > - return 0; > + return -1; > /* Yuck -- line ought to be "const char *"! */ > hold = line[len]; > line[len] = '\0'; > -- > 1.7.4.74.g639db > > -- Duy -- 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