Skip to content

Commit fd99b36

Browse files
committed
mailmap: fix bogus for() loop that happened to be safe by accident
The empty loop pretended to have an empty statement as its body by a phony indentation, but in fact was slurping the next statement into it. Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent 384b32c commit fd99b36

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

mailmap.c

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,10 @@ int read_mailmap(struct path_list *map, const char *filename, char **repo_abbrev
4242
continue;
4343
if (right_bracket == left_bracket + 1)
4444
continue;
45-
for (end_of_name = left_bracket; end_of_name != buffer
46-
&& isspace(end_of_name[-1]); end_of_name--)
47-
/* keep on looking */
45+
for (end_of_name = left_bracket;
46+
end_of_name != buffer && isspace(end_of_name[-1]);
47+
end_of_name--)
48+
; /* keep on looking */
4849
if (end_of_name == buffer)
4950
continue;
5051
name = xmalloc(end_of_name - buffer + 1);

0 commit comments

Comments
 (0)