On Mon, Sep 20 2021, Jeff King wrote: > -static int patmatch(struct grep_pat *p, char *line, char *eol, > +static int patmatch(struct grep_pat *p, > + const char *line, const char *eol, > regmatch_t *match, int eflags) nit; not worth a re-roll, over-line-wrapping here? You usually stay at column 79. In this case both line and eol would fit on the patmatch() lin, which is being changed anyway...