From: Jean Delvare Subject: Drop useless test in another_hunk() Upstream: Submitted (2014-11-10) This test will always succeed so it is either broken or useless. The equivalent code path for context patches doesn't have this test so I suppose it's OK to just remove it. --- src/pch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/src/pch.c +++ b/src/pch.c @@ -1651,7 +1651,7 @@ another_hunk (enum diff difftype, bool r if (*s == ' ') s++; if (*s++ != '@') malformed (); - if (*s++ == '@' && *s == ' ' && *s != '\0') + if (*s++ == '@' && *s == ' ') { p_c_function = s; while (*s != '\n')