Mark Wooding <mdw@xxxxxxxxxxxxxxxx> writes:
> The function names are parsed by a particularly stupid algorithm at the
> moment: it just tries to find a line in the `old' file, from before the
> start of the hunk, whose first character looks plausible. Still, it's
> most definitely a start.
> + (isalpha((unsigned char)*rec) || /* identifier? */
> + *rec == '_' || /* also identifier? */
> + *rec == '(' || /* lisp defun? */
> + *rec == '#')) { /* #define? */
GNU diff -p does "^[[:alpha:]$_]"; personally I think any line
that does not begin with a whitespace is good enough. In either
way, your patch is good. Thanks.
-
: 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