From: Junio C Hamano <[email protected]>
>
> Is this from the same Christian?
Yes, ...
> The series seems to have unusually high rate of style violations
> compared to the usual submission, like these:
>
> ERROR: open brace '{' following function declarations go on the next line
> #78: FILE: trailer.c:44:
> +static size_t alnum_len(const char *buf, size_t len) {
>
> ERROR: trailing statements should be on next line
> #79: FILE: trailer.c:45:
> + while (--len >= 0 && !isalnum(buf[len]));
>
> ERROR: space required before the open parenthesis '('
> #70: FILE: trailer.c:90:
> + switch(arg_tok->conf->if_exist) {
>
> WARNING: braces {} are not necessary for any arm of this statement
> #66: FILE: trailer.c:270:
> + if (!strcasecmp("doNothing", value)) {
> [...]
> + } else if (!strcasecmp("add", value)) {
> [...]
> + } else
> [...]
>
> ERROR: that open brace { should be on the previous line
> #96: FILE: trailer.c:300:
> + for (previous = NULL, item = first_conf_item;
> + item;
> + previous = item, item = item->next)
> + {
>
> Just trying to see if there is an impersonation ;-)
... I guess the artistic part of my mind is trying to come out somehow.
Thanks,
Christian.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
