> + /*
> + * If there is no command ready to run, should return directly to destroy
> + * temporary data in the quarantine area.
> + */
> + for (cmd = commands; cmd && cmd->error_string; cmd = cmd->next);
Write the empty body of the loop like this:
for (...)
; /* nothing */
to make it stand out.
> + if (!cmd)
> + return;
> +
> -mk_empty () {
> +mk_empty() {
> repo_name="$1"
> rm -fr "$repo_name" &&
> - mkdir "$repo_name" &&
> - (
> - cd "$repo_name" &&
> - git init &&
> - git config receive.denyCurrentBranch warn &&
> - mv .git/hooks .git/hooks-disabled
> - )
> + mkdir "$repo_name" &&
> + (
> + cd "$repo_name" &&
> + git init &&
> + git config receive.denyCurrentBranch warn &&
> + mv .git/hooks .git/hooks-disabled
> + )
> }
Documentation/CodingGuidelines. As far as I can tell, the above
does not change anything the function does, and the only change in
the patch is to violate the style guide badly. Why?
