{"thread":{"id":"33756","subject":"[PATCH 4/4] fast-export: trivial cleanup","startedAt":"2013-05-09T01:16:54Z","lastAt":"2013-05-17T17:18:32Z","messageCount":40,"participants":["Felipe Contreras","John Szakmeister","Ramkumar Ramachandra","Junio C Hamano","Matthieu Moy"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"216737","messageId":"1368062218-22440-1-git-send-email-felipe.contreras@gmail.com","threadId":"33756","inReplyTo":null,"subject":"[PATCH 0/4] trivial patches","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T01:16:54Z","receivedAt":"2013-05-09T01:16:54Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Hi,\n\nHere's a bunch of trivial style cleanup patches.\n\nFelipe Contreras (4):\n  documentation: trivial style cleanups\n  transport-helper: trivial style cleanup\n  {fast-export,transport-helper}: style cleanups\n  fast-export: trivial cleanup\n\n Documentation/git-am.txt               |  7 +++----\n Documentation/git-archive.txt          |  2 +-\n Documentation/git-check-ref-format.txt |  3 +--\n Documentation/git-clone.txt            |  4 ++--\n Documentation/git-commit.txt           |  2 +-\n Documentation/git-config.txt           |  3 +--\n Documentation/git-daemon.txt           |  5 ++---\n Documentation/git-difftool.txt         |  3 +--\n Documentation/git-fast-export.txt      | 10 +++++-----\n Documentation/git-fetch-pack.txt       |  6 +++---\n Documentation/git-fmt-merge-msg.txt    |  3 +--\n Documentation/git-fsck.txt             |  6 ++----\n Documentation/git-grep.txt             |  2 +-\n Documentation/git-mailinfo.txt         |  2 +-\n Documentation/git-merge.txt            |  3 +--\n Documentation/git-mergetool.txt        |  2 +-\n Documentation/git-push.txt             |  3 +--\n Documentation/git-remote.txt           |  2 +-\n Documentation/git-revert.txt           |  2 +-\n Documentation/git-update-index.txt     | 10 ++++------\n builtin/fast-export.c                  | 29 +++++++++++++----------------\n transport-helper.c                     |  3 +--\n 22 files changed, 48 insertions(+), 64 deletions(-)\n\n-- \n1.8.3.rc1.553.gac13664\n"},{"id":"216739","messageId":"1368062218-22440-2-git-send-email-felipe.contreras@gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 1/4] documentation: trivial style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T01:16:55Z","receivedAt":"2013-05-09T01:16:55Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"White-spaces, missing braces, standardize --[no-]foo.\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n Documentation/git-am.txt               |  7 +++----\n Documentation/git-archive.txt          |  2 +-\n Documentation/git-check-ref-format.txt |  3 +--\n Documentation/git-clone.txt            |  4 ++--\n Documentation/git-commit.txt           |  2 +-\n Documentation/git-config.txt           |  3 +--\n Documentation/git-daemon.txt           |  5 ++---\n Documentation/git-difftool.txt         |  3 +--\n Documentation/git-fast-export.txt      | 10 +++++-----\n Documentation/git-fetch-pack.txt       |  6 +++---\n Documentation/git-fmt-merge-msg.txt    |  3 +--\n Documentation/git-fsck.txt             |  6 ++----\n Documentation/git-grep.txt             |  2 +-\n Documentation/git-mailinfo.txt         |  2 +-\n Documentation/git-merge.txt            |  3 +--\n Documentation/git-mergetool.txt        |  2 +-\n Documentation/git-push.txt             |  3 +--\n Documentation/git-remote.txt           |  2 +-\n Documentation/git-revert.txt           |  2 +-\n Documentation/git-update-index.txt     | 10 ++++------\n 20 files changed, 34 insertions(+), 46 deletions(-)\n\ndiff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\nindex 19d57a8..5bbe7b6 100644\n--- a/Documentation/git-am.txt\n+++ b/Documentation/git-am.txt\n@@ -9,12 +9,12 @@ git-am - Apply a series of patches from a mailbox\n SYNOPSIS\n --------\n [verse]\n-'git am' [--signoff] [--keep] [--keep-cr | --no-keep-cr] [--utf8 | --no-utf8]\n+'git am' [--signoff] [--keep] [--[no-]keep-cr] [--[no-]utf8]\n \t [--3way] [--interactive] [--committer-date-is-author-date]\n \t [--ignore-date] [--ignore-space-change | --ignore-whitespace]\n \t [--whitespace=<option>] [-C<n>] [-p<n>] [--directory=<dir>]\n \t [--exclude=<path>] [--include=<path>] [--reject] [-q | --quiet]\n-\t [--scissors | --no-scissors]\n+\t [--[no-]scissors]\n \t [(<mbox> | <Maildir>)...]\n 'git am' (--continue | --skip | --abort)\n \n@@ -43,8 +43,7 @@ OPTIONS\n --keep-non-patch::\n \tPass `-b` flag to 'git mailinfo' (see linkgit:git-mailinfo[1]).\n \n---keep-cr::\n---no-keep-cr::\n+--[no-]keep-cr::\n \tWith `--keep-cr`, call 'git mailsplit' (see linkgit:git-mailsplit[1])\n \twith the same option, to prevent it from stripping CR at the end of\n \tlines. `am.keepcr` configuration variable can be used to specify the\ndiff --git a/Documentation/git-archive.txt b/Documentation/git-archive.txt\nindex 250e522..b97aaab 100644\n--- a/Documentation/git-archive.txt\n+++ b/Documentation/git-archive.txt\n@@ -10,7 +10,7 @@ SYNOPSIS\n --------\n [verse]\n 'git archive' [--format=<fmt>] [--list] [--prefix=<prefix>/] [<extra>]\n-\t      [-o | --output=<file>] [--worktree-attributes]\n+\t      [-o <file> | --output=<file>] [--worktree-attributes]\n \t      [--remote=<repo> [--exec=<git-upload-archive>]] <tree-ish>\n \t      [<path>...]\n \ndiff --git a/Documentation/git-check-ref-format.txt b/Documentation/git-check-ref-format.txt\nindex ec1739a..a49be1b 100644\n--- a/Documentation/git-check-ref-format.txt\n+++ b/Documentation/git-check-ref-format.txt\n@@ -83,8 +83,7 @@ typed the branch name.\n \n OPTIONS\n -------\n---allow-onelevel::\n---no-allow-onelevel::\n+--[no-]allow-onelevel::\n \tControls whether one-level refnames are accepted (i.e.,\n \trefnames that do not contain multiple `/`-separated\n \tcomponents).  The default is `--no-allow-onelevel`.\ndiff --git a/Documentation/git-clone.txt b/Documentation/git-clone.txt\nindex 5c16e31..a0727d7 100644\n--- a/Documentation/git-clone.txt\n+++ b/Documentation/git-clone.txt\n@@ -14,7 +14,7 @@ SYNOPSIS\n \t  [-o <name>] [-b <name>] [-u <upload-pack>] [--reference <repository>]\n \t  [--separate-git-dir <git dir>]\n \t  [--depth <depth>] [--[no-]single-branch]\n-\t  [--recursive|--recurse-submodules] [--] <repository>\n+\t  [--recursive | --recurse-submodules] [--] <repository>\n \t  [<directory>]\n \n DESCRIPTION\n@@ -188,7 +188,7 @@ objects from the source repository into a pack in the cloned repository.\n \twith a long history, and would want to send in fixes\n \tas patches.\n \n---single-branch::\n+--[no-]single-branch::\n \tClone only the history leading to the tip of a single branch,\n \teither specified by the `--branch` option or the primary\n \tbranch remote's `HEAD` points at. When creating a shallow\ndiff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt\nindex 8172938..1a7616c 100644\n--- a/Documentation/git-commit.txt\n+++ b/Documentation/git-commit.txt\n@@ -12,7 +12,7 @@ SYNOPSIS\n \t   [--dry-run] [(-c | -C | --fixup | --squash) <commit>]\n \t   [-F <file> | -m <msg>] [--reset-author] [--allow-empty]\n \t   [--allow-empty-message] [--no-verify] [-e] [--author=<author>]\n-\t   [--date=<date>] [--cleanup=<mode>] [--status | --no-status]\n+\t   [--date=<date>] [--cleanup=<mode>] [--[no-]status]\n \t   [-i | -o] [-S[<keyid>]] [--] [<file>...]\n \n DESCRIPTION\ndiff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\nindex 9ae2508..d88a6fc 100644\n--- a/Documentation/git-config.txt\n+++ b/Documentation/git-config.txt\n@@ -186,8 +186,7 @@ See also <<FILES>>.\n \tOpens an editor to modify the specified config file; either\n \t'--system', '--global', or repository (default).\n \n---includes::\n---no-includes::\n+--[no-]includes::\n \tRespect `include.*` directives in config files when looking up\n \tvalues. Defaults to on.\n \ndiff --git a/Documentation/git-daemon.txt b/Documentation/git-daemon.txt\nindex bfb106c..61a5701 100644\n--- a/Documentation/git-daemon.txt\n+++ b/Documentation/git-daemon.txt\n@@ -17,7 +17,7 @@ SYNOPSIS\n \t     [--enable=<service>] [--disable=<service>]\n \t     [--allow-override=<service>] [--forbid-override=<service>]\n \t     [--access-hook=<path>]\n-\t     [--inetd | [--listen=<host_or_ipaddr>] [--port=<n>] [--user=<user> [--group=<group>]]\n+\t     [--inetd | [--listen=<host_or_ipaddr>] [--port=<n>] [--user=<user> [--group=<group>]]]\n \t     [<directory>...]\n \n DESCRIPTION\n@@ -169,8 +169,7 @@ Git configuration files in that directory are readable by `<user>`.\n \trepository configuration.  By default, all the services\n \tare overridable.\n \n---informative-errors::\n---no-informative-errors::\n+--[no-]informative-errors::\n \tWhen informative errors are turned on, git-daemon will report\n \tmore verbose errors to the client, differentiating conditions\n \tlike \"no such repository\" from \"repository not exported\". This\ndiff --git a/Documentation/git-difftool.txt b/Documentation/git-difftool.txt\nindex 8361e6e..11887e6 100644\n--- a/Documentation/git-difftool.txt\n+++ b/Documentation/git-difftool.txt\n@@ -69,8 +69,7 @@ with custom merge tool commands and has the same value as `$MERGED`.\n --tool-help::\n \tPrint a list of diff tools that may be used with `--tool`.\n \n---symlinks::\n---no-symlinks::\n+--[no-]symlinks::\n \t'git difftool''s default behavior is create symlinks to the\n \tworking tree when run in `--dir-diff` mode and the right-hand\n \tside of the comparison yields the same content as the file in\ndiff --git a/Documentation/git-fast-export.txt b/Documentation/git-fast-export.txt\nindex 03fc8c3..efb0380 100644\n--- a/Documentation/git-fast-export.txt\n+++ b/Documentation/git-fast-export.txt\n@@ -106,11 +106,11 @@ marks the same across runs.\n \tdifferent from the commit's first parent).\n \n [<git-rev-list-args>...]::\n-       A list of arguments, acceptable to 'git rev-parse' and\n-       'git rev-list', that specifies the specific objects and references\n-       to export.  For example, `master~10..master` causes the\n-       current master reference to be exported along with all objects\n-       added since its 10th ancestor commit.\n+\tA list of arguments, acceptable to 'git rev-parse' and\n+\t'git rev-list', that specifies the specific objects and references\n+\tto export.  For example, `master~10..master` causes the\n+\tcurrent master reference to be exported along with all objects\n+\tadded since its 10th ancestor commit.\n \n EXAMPLES\n --------\ndiff --git a/Documentation/git-fetch-pack.txt b/Documentation/git-fetch-pack.txt\nindex b81e90d..1e71754 100644\n--- a/Documentation/git-fetch-pack.txt\n+++ b/Documentation/git-fetch-pack.txt\n@@ -10,9 +10,9 @@ SYNOPSIS\n --------\n [verse]\n 'git fetch-pack' [--all] [--quiet|-q] [--keep|-k] [--thin] [--include-tag]\n-\t\t\t\t[--upload-pack=<git-upload-pack>]\n-\t\t\t\t[--depth=<n>] [--no-progress]\n-\t\t\t\t[-v] [<host>:]<directory> [<refs>...]\n+\t[--upload-pack=<git-upload-pack>]\n+\t[--depth=<n>] [--no-progress]\n+\t[-v] [<host>:]<directory> [<refs>...]\n \n DESCRIPTION\n -----------\ndiff --git a/Documentation/git-fmt-merge-msg.txt b/Documentation/git-fmt-merge-msg.txt\nindex 3a0f55e..bb1232a 100644\n--- a/Documentation/git-fmt-merge-msg.txt\n+++ b/Documentation/git-fmt-merge-msg.txt\n@@ -35,8 +35,7 @@ OPTIONS\n \tDo not list one-line descriptions from the actual commits being\n \tmerged.\n \n---summary::\n---no-summary::\n+--[no-]summary::\n \tSynonyms to --log and --no-log; these are deprecated and will be\n \tremoved in the future.\n \ndiff --git a/Documentation/git-fsck.txt b/Documentation/git-fsck.txt\nindex e5878bd..25c431d 100644\n--- a/Documentation/git-fsck.txt\n+++ b/Documentation/git-fsck.txt\n@@ -30,8 +30,7 @@ index file, all SHA-1 references in `refs` namespace, and all reflogs\n \tPrint out objects that exist but that aren't reachable from any\n \tof the reference nodes.\n \n---dangling::\n---no-dangling::\n+--[no-]dangling::\n \tPrint objects that exist but that are never 'directly' used (default).\n \t`--no-dangling` can be used to omit this information from the output.\n \n@@ -78,8 +77,7 @@ index file, all SHA-1 references in `refs` namespace, and all reflogs\n \ta blob, the contents are written into the file, rather than\n \tits object name.\n \n---progress::\n---no-progress::\n+--[no-]progress::\n \tProgress status is reported on the standard error stream by\n \tdefault when it is attached to a terminal, unless\n \t--no-progress or --verbose is specified. --progress forces\ndiff --git a/Documentation/git-grep.txt b/Documentation/git-grep.txt\nindex 50d46e1..8497aa4 100644\n--- a/Documentation/git-grep.txt\n+++ b/Documentation/git-grep.txt\n@@ -25,7 +25,7 @@ SYNOPSIS\n \t   [-W | --function-context]\n \t   [-f <file>] [-e] <pattern>\n \t   [--and|--or|--not|(|)|-e <pattern>...]\n-\t   [ [--exclude-standard] [--cached | --no-index | --untracked] | <tree>...]\n+\t   [ [--[no-]exclude-standard] [--cached | --no-index | --untracked] | <tree>...]\n \t   [--] [<pathspec>...]\n \n DESCRIPTION\ndiff --git a/Documentation/git-mailinfo.txt b/Documentation/git-mailinfo.txt\nindex 97e7a8e..164a3c6 100644\n--- a/Documentation/git-mailinfo.txt\n+++ b/Documentation/git-mailinfo.txt\n@@ -9,7 +9,7 @@ git-mailinfo - Extracts patch and authorship from a single e-mail message\n SYNOPSIS\n --------\n [verse]\n-'git mailinfo' [-k|-b] [-u | --encoding=<encoding> | -n] [--scissors] <msg> <patch>\n+'git mailinfo' [-k|-b] [-u | --encoding=<encoding> | -n] [--[no-]scissors] <msg> <patch>\n \n \n DESCRIPTION\ndiff --git a/Documentation/git-merge.txt b/Documentation/git-merge.txt\nindex 42391f2..67ca99c 100644\n--- a/Documentation/git-merge.txt\n+++ b/Documentation/git-merge.txt\n@@ -76,8 +76,7 @@ The 'git fmt-merge-msg' command can be\n used to give a good default for automated 'git merge'\n invocations.\n \n---rerere-autoupdate::\n---no-rerere-autoupdate::\n+--[no-]rerere-autoupdate::\n \tAllow the rerere mechanism to update the index with the\n \tresult of auto-conflict resolution if possible.\n \ndiff --git a/Documentation/git-mergetool.txt b/Documentation/git-mergetool.txt\nindex 6b563c5..d758f3a 100644\n--- a/Documentation/git-mergetool.txt\n+++ b/Documentation/git-mergetool.txt\n@@ -8,7 +8,7 @@ git-mergetool - Run merge conflict resolution tools to resolve merge conflicts\n SYNOPSIS\n --------\n [verse]\n-'git mergetool' [--tool=<tool>] [-y|--no-prompt|--prompt] [<file>...]\n+'git mergetool' [--tool=<tool>] [-y|--[no-]prompt] [<file>...]\n \n DESCRIPTION\n -----------\ndiff --git a/Documentation/git-push.txt b/Documentation/git-push.txt\nindex eb2883c..d514813 100644\n--- a/Documentation/git-push.txt\n+++ b/Documentation/git-push.txt\n@@ -162,8 +162,7 @@ useful if you write an alias or script around 'git push'.\n \tlinkgit:git-pull[1] and other commands. For more information,\n \tsee 'branch.<name>.merge' in linkgit:git-config[1].\n \n---thin::\n---no-thin::\n+--[no-]thin::\n \tThese options are passed to linkgit:git-send-pack[1]. A thin transfer\n \tsignificantly reduces the amount of sent data when the sender and\n \treceiver share many of the same objects in common. The default is\ndiff --git a/Documentation/git-remote.txt b/Documentation/git-remote.txt\nindex 7a6f354..581bb4c 100644\n--- a/Documentation/git-remote.txt\n+++ b/Documentation/git-remote.txt\n@@ -10,7 +10,7 @@ SYNOPSIS\n --------\n [verse]\n 'git remote' [-v | --verbose]\n-'git remote add' [-t <branch>] [-m <master>] [-f] [--tags|--no-tags] [--mirror=<fetch|push>] <name> <url>\n+'git remote add' [-t <branch>] [-m <master>] [-f] [--[no-]tags] [--mirror=<fetch|push>] <name> <url>\n 'git remote rename' <old> <new>\n 'git remote remove' <name>\n 'git remote set-head' <name> (-a | -d | <branch>)\ndiff --git a/Documentation/git-revert.txt b/Documentation/git-revert.txt\nindex 70152e8..f79c9d8 100644\n--- a/Documentation/git-revert.txt\n+++ b/Documentation/git-revert.txt\n@@ -8,7 +8,7 @@ git-revert - Revert some existing commits\n SYNOPSIS\n --------\n [verse]\n-'git revert' [--edit | --no-edit] [-n] [-m parent-number] [-s] <commit>...\n+'git revert' [--[no-]edit] [-n] [-m parent-number] [-s] <commit>...\n 'git revert' --continue\n 'git revert' --quit\n 'git revert' --abort\ndiff --git a/Documentation/git-update-index.txt b/Documentation/git-update-index.txt\nindex 670e9fb..e0a8702 100644\n--- a/Documentation/git-update-index.txt\n+++ b/Documentation/git-update-index.txt\n@@ -14,8 +14,8 @@ SYNOPSIS\n \t     [--refresh] [-q] [--unmerged] [--ignore-missing]\n \t     [(--cacheinfo <mode> <object> <file>)...]\n \t     [--chmod=(+|-)x]\n-\t     [--assume-unchanged | --no-assume-unchanged]\n-\t     [--skip-worktree | --no-skip-worktree]\n+\t     [--[no-]assume-unchanged]\n+\t     [--[no-]skip-worktree]\n \t     [--ignore-submodules]\n \t     [--really-refresh] [--unresolve] [--again | -g]\n \t     [--info-only] [--index-info]\n@@ -77,8 +77,7 @@ OPTIONS\n --chmod=(+|-)x::\n         Set the execute permissions on the updated files.\n \n---assume-unchanged::\n---no-assume-unchanged::\n+--[no-]assume-unchanged::\n \tWhen these flags are specified, the object names recorded\n \tfor the paths are not updated.  Instead, these options\n \tset and unset the \"assume unchanged\" bit for the\n@@ -102,8 +101,7 @@ you will need to handle the situation manually.\n \tLike '--refresh', but checks stat information unconditionally,\n \twithout regard to the \"assume unchanged\" setting.\n \n---skip-worktree::\n---no-skip-worktree::\n+--[no-]skip-worktree::\n \tWhen one of these flags is specified, the object name recorded\n \tfor the paths are not updated. Instead, these options\n \tset and unset the \"skip-worktree\" bit for the paths. See\n-- \n1.8.3.rc1.553.gac13664\n"},{"id":"216735","messageId":"1368062218-22440-3-git-send-email-felipe.contreras@gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 2/4] transport-helper: trivial style cleanup","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T01:16:56Z","receivedAt":"2013-05-09T01:16:56Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n transport-helper.c | 3 +--\n 1 file changed, 1 insertion(+), 2 deletions(-)\n\ndiff --git a/transport-helper.c b/transport-helper.c\nindex 835815f..522d791 100644\n--- a/transport-helper.c\n+++ b/transport-helper.c\n@@ -214,9 +214,8 @@ static struct child_process *get_helper(struct transport *transport)\n \t\tint i;\n \t\tdata->refspec_nr = refspec_nr;\n \t\tdata->refspecs = parse_fetch_refspec(refspec_nr, refspecs);\n-\t\tfor (i = 0; i < refspec_nr; i++) {\n+\t\tfor (i = 0; i < refspec_nr; i++)\n \t\t\tfree((char *)refspecs[i]);\n-\t\t}\n \t\tfree(refspecs);\n \t}\n \tstrbuf_release(&buf);\n-- \n1.8.3.rc1.553.gac13664\n"},{"id":"216736","messageId":"1368062218-22440-4-git-send-email-felipe.contreras@gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T01:16:57Z","receivedAt":"2013-05-09T01:16:57Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n builtin/fast-export.c | 24 ++++++++++++------------\n 1 file changed, 12 insertions(+), 12 deletions(-)\n\ndiff --git a/builtin/fast-export.c b/builtin/fast-export.c\nindex d60d675..8091354 100644\n--- a/builtin/fast-export.c\n+++ b/builtin/fast-export.c\n@@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n \n \tbuf = read_sha1_file(sha1, &type, &size);\n \tif (!buf)\n-\t\tdie (\"Could not read blob %s\", sha1_to_hex(sha1));\n+\t\tdie(\"Could not read blob %s\", sha1_to_hex(sha1));\n \tif (check_sha1_signature(sha1, buf, size, typename(type)) < 0)\n \t\tdie(\"sha1 mismatch in blob %s\", sha1_to_hex(sha1));\n \tobject = parse_object_buffer(sha1, type, size, buf, &eaten);\n@@ -146,7 +146,7 @@ static void export_blob(const unsigned char *sha1)\n \n \tprintf(\"blob\\nmark :%\"PRIu32\"\\ndata %lu\\n\", last_idnum, size);\n \tif (size && fwrite(buf, size, 1, stdout) != 1)\n-\t\tdie_errno (\"Could not write blob '%s'\", sha1_to_hex(sha1));\n+\t\tdie_errno(\"Could not write blob '%s'\", sha1_to_hex(sha1));\n \tprintf(\"\\n\");\n \n \tshow_progress();\n@@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n \tparse_commit(commit);\n \tauthor = strstr(commit->buffer, \"\\nauthor \");\n \tif (!author)\n-\t\tdie (\"Could not find author in commit %s\",\n+\t\tdie(\"Could not find author in commit %s\",\n \t\t     sha1_to_hex(commit->object.sha1));\n \tauthor++;\n \tauthor_end = strchrnul(author, '\\n');\n \tcommitter = strstr(author_end, \"\\ncommitter \");\n \tif (!committer)\n-\t\tdie (\"Could not find committer in commit %s\",\n+\t\tdie(\"Could not find committer in commit %s\",\n \t\t     sha1_to_hex(commit->object.sha1));\n \tcommitter++;\n \tcommitter_end = strchrnul(committer, '\\n');\n@@ -395,7 +395,7 @@ static void handle_tag(const char *name, struct tag *tag)\n \n \tbuf = read_sha1_file(tag->object.sha1, &type, &size);\n \tif (!buf)\n-\t\tdie (\"Could not read tag %s\", sha1_to_hex(tag->object.sha1));\n+\t\tdie(\"Could not read tag %s\", sha1_to_hex(tag->object.sha1));\n \tmessage = memmem(buf, size, \"\\n\\n\", 2);\n \tif (message) {\n \t\tmessage += 2;\n@@ -421,11 +421,11 @@ static void handle_tag(const char *name, struct tag *tag)\n \t\tif (signature)\n \t\t\tswitch(signed_tag_mode) {\n \t\t\tcase ABORT:\n-\t\t\t\tdie (\"Encountered signed tag %s; use \"\n+\t\t\t\tdie(\"Encountered signed tag %s; use \"\n \t\t\t\t     \"--signed-tags=<mode> to handle it.\",\n \t\t\t\t     sha1_to_hex(tag->object.sha1));\n \t\t\tcase WARN:\n-\t\t\t\twarning (\"Exporting signed tag %s\",\n+\t\t\t\twarning(\"Exporting signed tag %s\",\n \t\t\t\t\t sha1_to_hex(tag->object.sha1));\n \t\t\t\t/* fallthru */\n \t\t\tcase VERBATIM:\n@@ -446,7 +446,7 @@ static void handle_tag(const char *name, struct tag *tag)\n \tif (!tagged_mark) {\n \t\tswitch(tag_of_filtered_mode) {\n \t\tcase ABORT:\n-\t\t\tdie (\"Tag %s tags unexported object; use \"\n+\t\t\tdie(\"Tag %s tags unexported object; use \"\n \t\t\t     \"--tag-of-filtered-object=<mode> to handle it.\",\n \t\t\t     sha1_to_hex(tag->object.sha1));\n \t\tcase DROP:\n@@ -454,7 +454,7 @@ static void handle_tag(const char *name, struct tag *tag)\n \t\t\treturn;\n \t\tcase REWRITE:\n \t\t\tif (tagged->type != OBJ_COMMIT) {\n-\t\t\t\tdie (\"Tag %s tags unexported %s!\",\n+\t\t\t\tdie(\"Tag %s tags unexported %s!\",\n \t\t\t\t     sha1_to_hex(tag->object.sha1),\n \t\t\t\t     typename(tagged->type));\n \t\t\t}\n@@ -467,7 +467,7 @@ static void handle_tag(const char *name, struct tag *tag)\n \t\t\t\tif (!(p->object.flags & TREESAME))\n \t\t\t\t\tbreak;\n \t\t\t\tif (!p->parents)\n-\t\t\t\t\tdie (\"Can't find replacement commit for tag %s\\n\",\n+\t\t\t\t\tdie(\"Can't find replacement commit for tag %s\\n\",\n \t\t\t\t\t     sha1_to_hex(tag->object.sha1));\n \t\t\t\tp = p->parents->item;\n \t\t\t}\n@@ -677,7 +677,7 @@ int cmd_fast_export(int argc, const char **argv, const char *prefix)\n \t};\n \n \tif (argc == 1)\n-\t\tusage_with_options (fast_export_usage, options);\n+\t\tusage_with_options(fast_export_usage, options);\n \n \t/* we handle encodings */\n \tgit_config(git_default_config, NULL);\n@@ -689,7 +689,7 @@ int cmd_fast_export(int argc, const char **argv, const char *prefix)\n \targc = setup_revisions(argc, argv, &revs, NULL);\n \targc = parse_options(argc, argv, prefix, options, fast_export_usage, 0);\n \tif (argc > 1)\n-\t\tusage_with_options (fast_export_usage, options);\n+\t\tusage_with_options(fast_export_usage, options);\n \n \tif (use_done_feature)\n \t\tprintf(\"feature done\\n\");\n-- \n1.8.3.rc1.553.gac13664\n"},{"id":"216734","messageId":"1368062218-22440-5-git-send-email-felipe.contreras@gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-1-git-send-email-felipe.contreras@gmail.com","subject":"[PATCH 4/4] fast-export: trivial cleanup","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T01:16:58Z","receivedAt":"2013-05-09T01:16:58Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Cast the object to a commit, only to get the object back?\n\nSigned-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n---\n builtin/fast-export.c | 5 +----\n 1 file changed, 1 insertion(+), 4 deletions(-)\n\ndiff --git a/builtin/fast-export.c b/builtin/fast-export.c\nindex 8091354..d24b4d9 100644\n--- a/builtin/fast-export.c\n+++ b/builtin/fast-export.c\n@@ -550,7 +550,6 @@ static void get_tags_and_duplicates(struct rev_cmdline_info *info,\n \n static void handle_tags_and_duplicates(struct string_list *extra_refs)\n {\n-\tstruct commit *commit;\n \tint i;\n \n \tfor (i = extra_refs->nr - 1; i >= 0; i--) {\n@@ -562,9 +561,7 @@ static void handle_tags_and_duplicates(struct string_list *extra_refs)\n \t\t\tbreak;\n \t\tcase OBJ_COMMIT:\n \t\t\t/* create refs pointing to already seen commits */\n-\t\t\tcommit = (struct commit *)object;\n-\t\t\tprintf(\"reset %s\\nfrom :%d\\n\\n\", name,\n-\t\t\t       get_object_mark(&commit->object));\n+\t\t\tprintf(\"reset %s\\nfrom :%d\\n\\n\", name, get_object_mark(object));\n \t\t\tshow_progress();\n \t\t\tbreak;\n \t\t}\n-- \n1.8.3.rc1.553.gac13664\n"},{"id":"216751","messageId":"CAEBDL5XZhEo14WKiz2m3KFRX+NsTFhmcz3adSti33RATMd897w@mail.gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-4-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"John Szakmeister","fromEmail":"john@szakmeister.net","sentAt":"2013-05-09T08:46:32Z","receivedAt":"2013-05-09T08:46:32Z","isPatch":true,"sender":{"key":"john@szakmeister.net","avatar":"https://avatars.githubusercontent.com/u/448087?v=4"},"body":"On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n> ---\n>  builtin/fast-export.c | 24 ++++++++++++------------\n>  1 file changed, 12 insertions(+), 12 deletions(-)\n>\n> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n> index d60d675..8091354 100644\n> --- a/builtin/fast-export.c\n> +++ b/builtin/fast-export.c\n> @@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n[snip]\n> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>         parse_commit(commit);\n>         author = strstr(commit->buffer, \"\\nauthor \");\n>         if (!author)\n> -               die (\"Could not find author in commit %s\",\n> +               die(\"Could not find author in commit %s\",\n>                      sha1_to_hex(commit->object.sha1));\n\nIt looks like your simple replace didn't account for calls with\nmultiple lines.  Now the remaining lines don't line up.\n:-)  There's several more places like this in the patch.\n\n-John\n"},{"id":"216752","messageId":"CAMP44s1R4YmyCFEQVRwSH6x-nDO-1=vDcA02C7F86GJk0zkS6g@mail.gmail.com","threadId":"33756","inReplyTo":"CAEBDL5XZhEo14WKiz2m3KFRX+NsTFhmcz3adSti33RATMd897w@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T08:50:23Z","receivedAt":"2013-05-09T08:50:23Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 3:46 AM, John Szakmeister <john@szakmeister.net> wrote:\n> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n> <felipe.contreras@gmail.com> wrote:\n>> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n>> ---\n>>  builtin/fast-export.c | 24 ++++++++++++------------\n>>  1 file changed, 12 insertions(+), 12 deletions(-)\n>>\n>> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n>> index d60d675..8091354 100644\n>> --- a/builtin/fast-export.c\n>> +++ b/builtin/fast-export.c\n>> @@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n> [snip]\n>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>         parse_commit(commit);\n>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>         if (!author)\n>> -               die (\"Could not find author in commit %s\",\n>> +               die(\"Could not find author in commit %s\",\n>>                      sha1_to_hex(commit->object.sha1));\n>\n> It looks like your simple replace didn't account for calls with\n> multiple lines.  Now the remaining lines don't line up.\n> :-)  There's several more places like this in the patch.\n\nAFAIK neither the git or the Linux code-style specify how multiple\nlines with open parenthesis should align.\n\n-- \nFelipe Contreras\n"},{"id":"216753","messageId":"CAEBDL5VotNSZC_xHc3r49bBFtNwU+drWa_Pj=skC3xzk-CGpKA@mail.gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-5-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 4/4] fast-export: trivial cleanup","fromName":"John Szakmeister","fromEmail":"john@szakmeister.net","sentAt":"2013-05-09T08:52:15Z","receivedAt":"2013-05-09T08:52:15Z","isPatch":true,"sender":{"key":"john@szakmeister.net","avatar":"https://avatars.githubusercontent.com/u/448087?v=4"},"body":"On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n> Cast the object to a commit, only to get the object back?\n>\n> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n> ---\n>  builtin/fast-export.c | 5 +----\n>  1 file changed, 1 insertion(+), 4 deletions(-)\n>\n> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n> index 8091354..d24b4d9 100644\n> --- a/builtin/fast-export.c\n> +++ b/builtin/fast-export.c\n> @@ -550,7 +550,6 @@ static void get_tags_and_duplicates(struct rev_cmdline_info *info,\n>\n>  static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>  {\n> -       struct commit *commit;\n>         int i;\n>\n>         for (i = extra_refs->nr - 1; i >= 0; i--) {\n> @@ -562,9 +561,7 @@ static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>                         break;\n>                 case OBJ_COMMIT:\n>                         /* create refs pointing to already seen commits */\n> -                       commit = (struct commit *)object;\n> -                       printf(\"reset %s\\nfrom :%d\\n\\n\", name,\n> -                              get_object_mark(&commit->object));\n> +                       printf(\"reset %s\\nfrom :%d\\n\\n\", name, get_object_mark(object));\n\nFWIW, this line is now too long (exceeds 80 columns).  Good catch on\nthe casting though.\n\n-John\n\nPS  Sorry for the duplicate Felipe... I still need to get used to\nhitting \"Reply All\". :-)\n"},{"id":"216754","messageId":"CAMP44s1DPwfiEDuUqUSBN1ShsgLDy9_2E0OD6nnwT7kJeJUCyg@mail.gmail.com","threadId":"33756","inReplyTo":"CAEBDL5VotNSZC_xHc3r49bBFtNwU+drWa_Pj=skC3xzk-CGpKA@mail.gmail.com","subject":"Re: [PATCH 4/4] fast-export: trivial cleanup","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T08:53:13Z","receivedAt":"2013-05-09T08:53:13Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 3:52 AM, John Szakmeister <john@szakmeister.net> wrote:\n> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n> <felipe.contreras@gmail.com> wrote:\n>> Cast the object to a commit, only to get the object back?\n>>\n>> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n>> ---\n>>  builtin/fast-export.c | 5 +----\n>>  1 file changed, 1 insertion(+), 4 deletions(-)\n>>\n>> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n>> index 8091354..d24b4d9 100644\n>> --- a/builtin/fast-export.c\n>> +++ b/builtin/fast-export.c\n>> @@ -550,7 +550,6 @@ static void get_tags_and_duplicates(struct rev_cmdline_info *info,\n>>\n>>  static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>>  {\n>> -       struct commit *commit;\n>>         int i;\n>>\n>>         for (i = extra_refs->nr - 1; i >= 0; i--) {\n>> @@ -562,9 +561,7 @@ static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>>                         break;\n>>                 case OBJ_COMMIT:\n>>                         /* create refs pointing to already seen commits */\n>> -                       commit = (struct commit *)object;\n>> -                       printf(\"reset %s\\nfrom :%d\\n\\n\", name,\n>> -                              get_object_mark(&commit->object));\n>> +                       printf(\"reset %s\\nfrom :%d\\n\\n\", name, get_object_mark(object));\n>\n> FWIW, this line is now too long (exceeds 80 columns).  Good catch on\n> the casting though.\n>\n> -John\n>\n> PS  Sorry for the duplicate Felipe... I still need to get used to\n> hitting \"Reply All\". :-)\n\nThe guideline is:\n\n - We try to keep to at most 80 characters per line.\n\nThe key word being *try*.\n\n-- \nFelipe Contreras\n"},{"id":"216755","messageId":"CAEBDL5Ww6g8G-eM93ZFji3ZNEbKrpsTuibHkOk=aSnFapV0EEA@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s1DPwfiEDuUqUSBN1ShsgLDy9_2E0OD6nnwT7kJeJUCyg@mail.gmail.com","subject":"Re: [PATCH 4/4] fast-export: trivial cleanup","fromName":"John Szakmeister","fromEmail":"john@szakmeister.net","sentAt":"2013-05-09T09:04:55Z","receivedAt":"2013-05-09T09:04:55Z","isPatch":true,"sender":{"key":"john@szakmeister.net","avatar":"https://avatars.githubusercontent.com/u/448087?v=4"},"body":"On Thu, May 9, 2013 at 4:53 AM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n[snip]\n>>> @@ -562,9 +561,7 @@ static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>>>                         break;\n>>>                 case OBJ_COMMIT:\n>>>                         /* create refs pointing to already seen commits */\n>>> -                       commit = (struct commit *)object;\n>>> -                       printf(\"reset %s\\nfrom :%d\\n\\n\", name,\n>>> -                              get_object_mark(&commit->object));\n>>> +                       printf(\"reset %s\\nfrom :%d\\n\\n\", name, get_object_mark(object));\n>>\n>> FWIW, this line is now too long (exceeds 80 columns).  Good catch on\n>> the casting though.\n>>\n>> -John\n>>\n>> PS  Sorry for the duplicate Felipe... I still need to get used to\n>> hitting \"Reply All\". :-)\n>\n> The guideline is:\n>\n>  - We try to keep to at most 80 characters per line.\n>\n> The key word being *try*.\n\nI saw that, but you actively joined the lines, and there was no need\nto.  It didn't even require trying to keep it within 80 columns. :-)\n\n-John\n"},{"id":"216759","messageId":"CAEBDL5Vu+gi_1CcJio0igNS2HmMrZNVUvWp3zeqjAgT6C7xZ0A@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s1R4YmyCFEQVRwSH6x-nDO-1=vDcA02C7F86GJk0zkS6g@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"John Szakmeister","fromEmail":"john@szakmeister.net","sentAt":"2013-05-09T09:30:51Z","receivedAt":"2013-05-09T09:30:51Z","isPatch":true,"sender":{"key":"john@szakmeister.net","avatar":"https://avatars.githubusercontent.com/u/448087?v=4"},"body":"On Thu, May 9, 2013 at 4:50 AM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n> On Thu, May 9, 2013 at 3:46 AM, John Szakmeister <john@szakmeister.net> wrote:\n>> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n[snip]\n>>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>>         parse_commit(commit);\n>>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>>         if (!author)\n>>> -               die (\"Could not find author in commit %s\",\n>>> +               die(\"Could not find author in commit %s\",\n>>>                      sha1_to_hex(commit->object.sha1));\n>>\n>> It looks like your simple replace didn't account for calls with\n>> multiple lines.  Now the remaining lines don't line up.\n>> :-)  There's several more places like this in the patch.\n>\n> AFAIK neither the git or the Linux code-style specify how multiple\n> lines with open parenthesis should align.\n\nTrue, but I'm sure you know well that the Linux coding style tends to\nfollow the GNU coding style--with modifications--which lines up args\nwith the first character inside the paren\n(http://www.gnu.org/prep/standards/html_node/Formatting.html#Formatting-Your-Source-Code),\nand it's clear that's what the previous author had done.\n\nThe hope was that you let your search and replace run awry, and I was\njust pointing out that it did, so it could fixed it up.  *shrug*  I've\nhad little participation, so I don't consider my opinion to count for\nmuch on the matter.\n\n-John\n"},{"id":"216760","messageId":"CAMP44s3JE0wPJJy+PN2tzfAabrYb3oGHUkhCRnY52O8T77TGAg@mail.gmail.com","threadId":"33756","inReplyTo":"CAEBDL5Vu+gi_1CcJio0igNS2HmMrZNVUvWp3zeqjAgT6C7xZ0A@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T10:18:25Z","receivedAt":"2013-05-09T10:18:25Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 4:30 AM, John Szakmeister <john@szakmeister.net> wrote:\n> On Thu, May 9, 2013 at 4:50 AM, Felipe Contreras\n> <felipe.contreras@gmail.com> wrote:\n>> On Thu, May 9, 2013 at 3:46 AM, John Szakmeister <john@szakmeister.net> wrote:\n>>> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n> [snip]\n>>>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>>>         parse_commit(commit);\n>>>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>>>         if (!author)\n>>>> -               die (\"Could not find author in commit %s\",\n>>>> +               die(\"Could not find author in commit %s\",\n>>>>                      sha1_to_hex(commit->object.sha1));\n>>>\n>>> It looks like your simple replace didn't account for calls with\n>>> multiple lines.  Now the remaining lines don't line up.\n>>> :-)  There's several more places like this in the patch.\n>>\n>> AFAIK neither the git or the Linux code-style specify how multiple\n>> lines with open parenthesis should align.\n>\n> True, but I'm sure you know well that the Linux coding style tends to\n> follow the GNU coding style--with modifications--which lines up args\n> with the first character inside the paren\n> (http://www.gnu.org/prep/standards/html_node/Formatting.html#Formatting-Your-Source-Code),\n> and it's clear that's what the previous author had done.\n\nNo, I don't. I don't see how the GNU style has anything to do with the\nLinux style.\n\n-- \nFelipe Contreras\n"},{"id":"216761","messageId":"CALkWK0m3AG5iw1gCAAepmzJeZD_s9cm-eYC-cnSaVmn=FvSNag@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s1R4YmyCFEQVRwSH6x-nDO-1=vDcA02C7F86GJk0zkS6g@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2013-05-09T10:24:10Z","receivedAt":"2013-05-09T10:24:10Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Felipe Contreras wrote:\n> AFAIK neither the git or the Linux code-style specify how multiple\n> lines with open parenthesis should align.\n\ngit style inherits from linux style.  See Chapter 9 of\nDocumentation/CodingStyle in linux.git.  It has elisp snippets you can\nstick in your .emacs.\n"},{"id":"216762","messageId":"CAMP44s3LCmhvomr9WAhWVusR1=jihO4KigSOYkoL7i=D-nAMDA@mail.gmail.com","threadId":"33756","inReplyTo":"CALkWK0m3AG5iw1gCAAepmzJeZD_s9cm-eYC-cnSaVmn=FvSNag@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T11:06:37Z","receivedAt":"2013-05-09T11:06:37Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 5:24 AM, Ramkumar Ramachandra <artagnon@gmail.com> wrote:\n> Felipe Contreras wrote:\n>> AFAIK neither the git or the Linux code-style specify how multiple\n>> lines with open parenthesis should align.\n>\n> git style inherits from linux style.  See Chapter 9 of\n> Documentation/CodingStyle in linux.git.  It has elisp snippets you can\n> stick in your .emacs.\n\nI don't see anything in Chapter 9 mentioning the alignment of\nparenthesis. This has been discussed multiple times in the Linux\nmailing lists, nobody cares enough about it to add such alignment\nguidelines to the documentation.\n\n-- \nFelipe Contreras\n"},{"id":"216804","messageId":"7vbo8k9exd.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAMP44s1R4YmyCFEQVRwSH6x-nDO-1=vDcA02C7F86GJk0zkS6g@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-09T18:38:38Z","receivedAt":"2013-05-09T18:38:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n>>>         if (!author)\n>>> -               die (\"Could not find author in commit %s\",\n>>> +               die(\"Could not find author in commit %s\",\n>>>                      sha1_to_hex(commit->object.sha1));\n>>\n>> It looks like your simple replace didn't account for calls with\n>> multiple lines.  Now the remaining lines don't line up.\n>> :-)  There's several more places like this in the patch.\n>\n> AFAIK neither the git or the Linux code-style specify how multiple\n> lines with open parenthesis should align.\n\nThen the usual \"follow the style of surrounding code\" rule would\napply, no?  It is clear that the original wants to align the opening\ndq of \"Could...\"  and 's' at the beginning of sha1_to_hex().\n\nWhy is it so hard for you to say \"Good eyes, thanks\"?\n"},{"id":"216805","messageId":"7v7gj89e9o.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAEBDL5Ww6g8G-eM93ZFji3ZNEbKrpsTuibHkOk=aSnFapV0EEA@mail.gmail.com","subject":"Re: [PATCH 4/4] fast-export: trivial cleanup","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-09T18:52:51Z","receivedAt":"2013-05-09T18:52:51Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Szakmeister <john@szakmeister.net> writes:\n\n> On Thu, May 9, 2013 at 4:53 AM, Felipe Contreras\n> <felipe.contreras@gmail.com> wrote:\n> [snip]\n>>>> @@ -562,9 +561,7 @@ static void handle_tags_and_duplicates(struct string_list *extra_refs)\n>>>>                         break;\n>>>>                 case OBJ_COMMIT:\n>>>>                         /* create refs pointing to already seen commits */\n>>>> -                       commit = (struct commit *)object;\n>>>> -                       printf(\"reset %s\\nfrom :%d\\n\\n\", name,\n>>>> -                              get_object_mark(&commit->object));\n>>>> +                       printf(\"reset %s\\nfrom :%d\\n\\n\", name, get_object_mark(object));\n>>>\n>>> FWIW, this line is now too long (exceeds 80 columns).  Good catch on\n>>> the casting though.\n>> ...\n>> The key word being *try*.\n>\n> I saw that, but you actively joined the lines, and there was no need\n> to.  It didn't even require trying to keep it within 80 columns. :-)\n\nThat is somewhat a flawed argument.\n\nSometimes it makes it easier to have a single line that is slightly\nlonger than 80-col than split it across two lines.  \"Now one of the\narguments to this function invocation is much shorter.  The only\nreason the invocation originally was split into two lines was\nbecause its argument list was too long, so joining them into a\nsingle line _might_ make it easier to read.  Let's try to see how it\nreads.\" is a reasonable line of thought, and at that point, it\nstarts to require trying to keep it under 80-col.\n\nIn this particular case, however, I do not think that the single\nline is particularly easier to read, but I do not think the original\nis better, either.  The easiest to read would be more like this:\n\n                       printf(\"reset %s\\nfrom :%d\\n\\n\",\n                              name, get_object_mark(object));\n\nto show which argument corresponds to which %conversion [*1*]\n\n\n[Footnote]\n\n*1* When the format string has too many embedded LFs, showing it\n    like this:\n\n                       printf(\"reset %s\\n\"\n                              \"from :%d\\n\\n\",\n                              name, get_object_mark(object));\n\n    is even better, but I do not think it applies to this particular\n    case.\n"},{"id":"216809","messageId":"CAMP44s3XsVb6k1Oynw_y7yvV1fWaSDGquFoRNMcaN0yGD5_UNQ@mail.gmail.com","threadId":"33756","inReplyTo":"7vbo8k9exd.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T19:23:04Z","receivedAt":"2013-05-09T19:23:04Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 1:38 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>>>>         if (!author)\n>>>> -               die (\"Could not find author in commit %s\",\n>>>> +               die(\"Could not find author in commit %s\",\n>>>>                      sha1_to_hex(commit->object.sha1));\n>>>\n>>> It looks like your simple replace didn't account for calls with\n>>> multiple lines.  Now the remaining lines don't line up.\n>>> :-)  There's several more places like this in the patch.\n>>\n>> AFAIK neither the git or the Linux code-style specify how multiple\n>> lines with open parenthesis should align.\n>\n> Then the usual \"follow the style of surrounding code\" rule would\n> apply, no?\n\nThe surrounding code has all the open parentheses wrong for these\nfunctions, that doesn't mean the surrounding code is correct.\n\n> It is clear that the original wants to align the opening\n> dq of \"Could...\"  and 's' at the beginning of sha1_to_hex().\n>\n> Why is it so hard for you to say \"Good eyes, thanks\"?\n\nBecause I didn't miss it, it was deliberate from my side, if nobody\ncan be bothered to add a rule on the coding style guidelines, why\nshould I bother? As I said, this pops up from time to time in Linux\nmailing lists[1], but at the end of the day the CodingStyle doesn't\nspecify what should be the way to align these, so any discussion about\nwhether they are properly aligned or not is futile.\n\nIf it was up to me, I would use the alignment that makes more sense,\nwhich is to align two tabs always, this way, each time somebody\ndecides to change the first line, the rest of the lines don't need to\nchange, which is something I see through the CodingStyle; try to avoid\na style that would generate bigger diffs.\n\nBut if I did such alignment I might create even more discussion, so I\ndeliberately decided not to change the alignment at all, which still\nleaves the code in accordance with the CodingStyle.\n\n[1] http://article.gmane.org/gmane.linux.kernel/1407801\n\n-- \nFelipe Contreras\n"},{"id":"216831","messageId":"7vr4hf7o3a.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"1368062218-22440-3-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 2/4] transport-helper: trivial style cleanup","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-09T23:03:37Z","receivedAt":"2013-05-09T23:03:37Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thanks; no brainer.\n"},{"id":"216833","messageId":"7vehdf7nts.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"1368062218-22440-2-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 1/4] documentation: trivial style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-09T23:09:19Z","receivedAt":"2013-05-09T23:09:19Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> White-spaces, missing braces, standardize --[no-]foo.\n>\n> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n\nThis is uncomfortably big at this phase of the release cycle, but\nthanks anyway.\n\nBecause I didn't want to review this patch only to spot silly\nformatting mistakes that may break the documentation build (which I\ndidn't find any), I looked the entire files the patch touches, and\nmany of the comments below ended up being suggestions for a\nfollow-up work for people who would want to pick low-hanging fruits.\n\nHint, hint.\n\nThere however are a few things that should have been in this patch,\nthough.  I'll attach what I'd queue as \"SQUASH???\" on top of this\npatch at the bottom.\n\n> diff --git a/Documentation/git-am.txt b/Documentation/git-am.txt\n> index 19d57a8..5bbe7b6 100644\n> --- a/Documentation/git-am.txt\n> +++ b/Documentation/git-am.txt\n> @@ -9,12 +9,12 @@ git-am - Apply a series of patches from a mailbox\n>  SYNOPSIS\n>  --------\n>  [verse]\n> -'git am' [--signoff] [--keep] [--keep-cr | --no-keep-cr] [--utf8 | --no-utf8]\n> +'git am' [--signoff] [--keep] [--[no-]keep-cr] [--[no-]utf8]\n>  \t [--3way] [--interactive] [--committer-date-is-author-date]\n>  \t [--ignore-date] [--ignore-space-change | --ignore-whitespace]\n>  \t [--whitespace=<option>] [-C<n>] [-p<n>] [--directory=<dir>]\n>  \t [--exclude=<path>] [--include=<path>] [--reject] [-q | --quiet]\n> -\t [--scissors | --no-scissors]\n> +\t [--[no-]scissors]\n>  \t [(<mbox> | <Maildir>)...]\n>  'git am' (--continue | --skip | --abort)\n>  \n> @@ -43,8 +43,7 @@ OPTIONS\n>  --keep-non-patch::\n>  \tPass `-b` flag to 'git mailinfo' (see linkgit:git-mailinfo[1]).\n>  \n> ---keep-cr::\n> ---no-keep-cr::\n> +--[no-]keep-cr::\n>  \tWith `--keep-cr`, call 'git mailsplit' (see linkgit:git-mailsplit[1])\n>  \twith the same option, to prevent it from stripping CR at the end of\n>  \tlines. `am.keepcr` configuration variable can be used to specify the\n\nOK.\n\nWe still have two separate entries for \"--scissors/--no-scissors\"\nand \"--utf8/--no-utf8\" pairs in the description text.  Do they want\nto get united somehow?\n\n> diff --git a/Documentation/git-clone.txt b/Documentation/git-clone.txt\n> index 5c16e31..a0727d7 100644\n> --- a/Documentation/git-clone.txt\n> +++ b/Documentation/git-clone.txt\n> @@ -14,7 +14,7 @@ SYNOPSIS\n>  \t  [-o <name>] [-b <name>] [-u <upload-pack>] [--reference <repository>]\n>  \t  [--separate-git-dir <git dir>]\n>  \t  [--depth <depth>] [--[no-]single-branch]\n> -\t  [--recursive|--recurse-submodules] [--] <repository>\n> +\t  [--recursive | --recurse-submodules] [--] <repository>\n>  \t  [<directory>]\n>  \n>  DESCRIPTION\n> @@ -188,7 +188,7 @@ objects from the source repository into a pack in the cloned repository.\n>  \twith a long history, and would want to send in fixes\n>  \tas patches.\n>  \n> ---single-branch::\n> +--[no-]single-branch::\n>  \tClone only the history leading to the tip of a single branch,\n>  \teither specified by the `--branch` option or the primary\n>  \tbranch remote's `HEAD` points at. When creating a shallow\n\nOK.\n\nWe have \"--no-hardlinks\" without \"--hardlinks\".  Does the parser\naccept \"--no-no-hardlinks\"?  If so, we may want to error it out.\n\nI would omit the options \"--no-hardlinks\" and \"--shared\", and added\n\"--local={link,copy,shared}\" instead, as these two options only make\nsense when cloning from a local repository, if I were doing the UI\nfrom scratch today.\n\n> diff --git a/Documentation/git-commit.txt b/Documentation/git-commit.txt\n> index 8172938..1a7616c 100644\n> --- a/Documentation/git-commit.txt\n> +++ b/Documentation/git-commit.txt\n> @@ -12,7 +12,7 @@ SYNOPSIS\n>  \t   [--dry-run] [(-c | -C | --fixup | --squash) <commit>]\n>  \t   [-F <file> | -m <msg>] [--reset-author] [--allow-empty]\n>  \t   [--allow-empty-message] [--no-verify] [-e] [--author=<author>]\n> -\t   [--date=<date>] [--cleanup=<mode>] [--status | --no-status]\n> +\t   [--date=<date>] [--cleanup=<mode>] [--[no-]status]\n>  \t   [-i | -o] [-S[<keyid>]] [--] [<file>...]\n>  \n>  DESCRIPTION\n\nOK.\n\nWe have \"--no-verify\" without \"--verify\".  Does the parser accept\n\"--no-no-verify\"?  If so we may want to error it out.\n\n> diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt\n> index 9ae2508..d88a6fc 100644\n> --- a/Documentation/git-config.txt\n> +++ b/Documentation/git-config.txt\n> @@ -186,8 +186,7 @@ See also <<FILES>>.\n>  \tOpens an editor to modify the specified config file; either\n>  \t'--system', '--global', or repository (default).\n>  \n> ---includes::\n> ---no-includes::\n> +--[no-]includes::\n>  \tRespect `include.*` directives in config files when looking up\n>  \tvalues. Defaults to on.\n\nOK.\n\nThe SYNOPSIS section begins like this:\n\n    SYNOPSIS\n    --------\n    [verse]\n    'git config' [<file-option>] [type] [-z|--null] name [value [value_regex]]\n    ...\n\nOther parts of this patch seems to want to have SPs around vertical\nbars that shows choices, so \"[-z | --null]\" would be necessary as a\nfollow-up to bring more consistency.\n\nThe placeholders like \"type\", \"name\", \"value\", etc. are not inside\n<angle-brackets>, which may want to be fixed as well.\n\nIt is unclear what \"<file-option>\" is.  A few paragraphs in the\nDESCRIPTION section talk about --system/global/etc.; there should be\na sentence there to mention that exact term <file-option> to help\nthe reader make a connection, just like the paragraph that talks\nabout \"type\" uses that word to make it clear the word in SYNOPSIS is\nabout things like --int/bool/path.\n\n> diff --git a/Documentation/git-daemon.txt b/Documentation/git-daemon.txt\n> index bfb106c..61a5701 100644\n> --- a/Documentation/git-daemon.txt\n> +++ b/Documentation/git-daemon.txt\n> @@ -17,7 +17,7 @@ SYNOPSIS\n>  \t     [--enable=<service>] [--disable=<service>]\n>  \t     [--allow-override=<service>] [--forbid-override=<service>]\n>  \t     [--access-hook=<path>]\n> -\t     [--inetd | [--listen=<host_or_ipaddr>] [--port=<n>] [--user=<user> [--group=<group>]]\n> +\t     [--inetd | [--listen=<host_or_ipaddr>] [--port=<n>] [--user=<user> [--group=<group>]]]\n>  \t     [<directory>...]\n\nYou can say \"--inetd\".  Only when you do not say \"--inetd\", you can\nuse \"--listen\", \"--port\", etc.  Further, only when you say \"--user\",\nyou can also say \"--group\".  OK.\n\nAs this is governed by [verse], it would be more helpful to show\nthat logic not just with the nesting of [], but make it stand out\nwith a line break, perhaps like:\n\n        [--inetd |\n         [--listen=<host>] [--port=<n>]\n         [--user=<user> [--group=<group>]]]\n\n(I do not mind to see a single line for the second and third lines).\n\n> @@ -169,8 +169,7 @@ Git configuration files in that directory are readable by `<user>`.\n>  \trepository configuration.  By default, all the services\n>  \tare overridable.\n>  \n> ---informative-errors::\n> ---no-informative-errors::\n> +--[no-]informative-errors::\n>  \tWhen informative errors are turned on, git-daemon will report\n>  \tmore verbose errors to the client, differentiating conditions\n>  \tlike \"no such repository\" from \"repository not exported\". This\n\nOK.\n\nBy the way, this is missing from SYNOPSIS, unlike all the other\noptions to this command.\n\n> diff --git a/Documentation/git-difftool.txt b/Documentation/git-difftool.txt\n> index 8361e6e..11887e6 100644\n> --- a/Documentation/git-difftool.txt\n> +++ b/Documentation/git-difftool.txt\n> @@ -69,8 +69,7 @@ with custom merge tool commands and has the same value as `$MERGED`.\n>  --tool-help::\n>  \tPrint a list of diff tools that may be used with `--tool`.\n>  \n> ---symlinks::\n> ---no-symlinks::\n> +--[no-]symlinks::\n>  \t'git difftool''s default behavior is create symlinks to the\n>  \tworking tree when run in `--dir-diff` mode and the right-hand\n>  \tside of the comparison yields the same content as the file in\n\nOK.\n\nWe have \"--no-symlinks\" without \"--symlinks\".  Does the parser\naccept \"--no-no-symlinks\"?  If so, we may want to error it out.\n\n> diff --git a/Documentation/git-fast-export.txt b/Documentation/git-fast-export.txt\n> index 03fc8c3..efb0380 100644\n> --- a/Documentation/git-fast-export.txt\n> +++ b/Documentation/git-fast-export.txt\n> @@ -106,11 +106,11 @@ marks the same across runs.\n>  \tdifferent from the commit's first parent).\n>  \n>  [<git-rev-list-args>...]::\n> -       A list of arguments, acceptable to 'git rev-parse' and\n> -       'git rev-list', that specifies the specific objects and references\n> -       to export.  For example, `master~10..master` causes the\n> -       current master reference to be exported along with all objects\n> -       added since its 10th ancestor commit.\n> +\tA list of arguments, acceptable to 'git rev-parse' and\n> +\t'git rev-list', that specifies the specific objects and references\n> +\tto export.  For example, `master~10..master` causes the\n> +\tcurrent master reference to be exported along with all objects\n> +\tadded since its 10th ancestor commit.\n\nOK.\n\nWe have \"--no-data\" without \"--data\".  Does the parser accept\n\"--no-no-data\"?  If so, we may want to error it out.\n\n> diff --git a/Documentation/git-fetch-pack.txt b/Documentation/git-fetch-pack.txt\n> index b81e90d..1e71754 100644\n> --- a/Documentation/git-fetch-pack.txt\n> +++ b/Documentation/git-fetch-pack.txt\n> @@ -10,9 +10,9 @@ SYNOPSIS\n>  --------\n>  [verse]\n>  'git fetch-pack' [--all] [--quiet|-q] [--keep|-k] [--thin] [--include-tag]\n> -\t\t\t\t[--upload-pack=<git-upload-pack>]\n> -\t\t\t\t[--depth=<n>] [--no-progress]\n> -\t\t\t\t[-v] [<host>:]<directory> [<refs>...]\n> +\t[--upload-pack=<git-upload-pack>]\n> +\t[--depth=<n>] [--no-progress]\n> +\t[-v] [<host>:]<directory> [<refs>...]\n\nOK.\n\nWe have \"--no-progress\" without \"--progress\".  Does the parser\naccept \"--no-no-progress\"?  If so, we may want to error it out.\n\n> diff --git a/Documentation/git-mergetool.txt b/Documentation/git-mergetool.txt\n> index 6b563c5..d758f3a 100644\n> --- a/Documentation/git-mergetool.txt\n> +++ b/Documentation/git-mergetool.txt\n> @@ -8,7 +8,7 @@ git-mergetool - Run merge conflict resolution tools to resolve merge conflicts\n>  SYNOPSIS\n>  --------\n>  [verse]\n> -'git mergetool' [--tool=<tool>] [-y|--no-prompt|--prompt] [<file>...]\n> +'git mergetool' [--tool=<tool>] [-y|--[no-]prompt] [<file>...]\n>  \n>  DESCRIPTION\n>  -----------\n\nOther parts of this patch seems to want to have SPs around vertical\nbars that shows choices, so \"[-y | --[no-]prompt]\" would be needed,\nespecially given that this patch touches that same line.\n\n> diff --git a/Documentation/git-revert.txt b/Documentation/git-revert.txt\n> index 70152e8..f79c9d8 100644\n> --- a/Documentation/git-revert.txt\n> +++ b/Documentation/git-revert.txt\n> @@ -8,7 +8,7 @@ git-revert - Revert some existing commits\n>  SYNOPSIS\n>  --------\n>  [verse]\n> -'git revert' [--edit | --no-edit] [-n] [-m parent-number] [-s] <commit>...\n> +'git revert' [--[no-]edit] [-n] [-m parent-number] [-s] <commit>...\n>  'git revert' --continue\n>  'git revert' --quit\n>  'git revert' --abort\n\nOK.\n\nThe DESCRIPTION section only describes \"--no-edit\", which may need\nto be fixed.\n\n\n\n-- squash --\n\n    SQUASH???\n\n    Update the overlong SYNOPSIS section for git-daemon. As the patch\n    adds SPs around vertical bars for [choice1 | choice2], match the\n    line it touches in git-mergetool.txt to also do that.\n\ndiff --git a/Documentation/git-daemon.txt b/Documentation/git-daemon.txt\nindex 61a5701..223f731 100644\n--- a/Documentation/git-daemon.txt\n+++ b/Documentation/git-daemon.txt\n@@ -16,8 +16,10 @@ SYNOPSIS\n \t     [--reuseaddr] [--detach] [--pid-file=<file>]\n \t     [--enable=<service>] [--disable=<service>]\n \t     [--allow-override=<service>] [--forbid-override=<service>]\n-\t     [--access-hook=<path>]\n-\t     [--inetd | [--listen=<host_or_ipaddr>] [--port=<n>] [--user=<user> [--group=<group>]]]\n+\t     [--access-hook=<path>] [--[no-]informative-errors]\n+\t     [--inetd |\n+\t      [--listen=<host_or_ipaddr>] [--port=<n>]\n+\t      [--user=<user> [--group=<group>]]]\n \t     [<directory>...]\n \n DESCRIPTION\ndiff --git a/Documentation/git-mergetool.txt b/Documentation/git-mergetool.txt\nindex d758f3a..07137f2 100644\n--- a/Documentation/git-mergetool.txt\n+++ b/Documentation/git-mergetool.txt\n@@ -8,7 +8,7 @@ git-mergetool - Run merge conflict resolution tools to resolve merge conflicts\n SYNOPSIS\n --------\n [verse]\n-'git mergetool' [--tool=<tool>] [-y|--[no-]prompt] [<file>...]\n+'git mergetool' [--tool=<tool>] [-y | --[no-]prompt] [<file>...]\n \n DESCRIPTION\n -----------\n"},{"id":"216834","messageId":"7v7gj77nt9.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAEBDL5XZhEo14WKiz2m3KFRX+NsTFhmcz3adSti33RATMd897w@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-09T23:09:38Z","receivedAt":"2013-05-09T23:09:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Szakmeister <john@szakmeister.net> writes:\n\n> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n> <felipe.contreras@gmail.com> wrote:\n>> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n>> ---\n>>  builtin/fast-export.c | 24 ++++++++++++------------\n>>  1 file changed, 12 insertions(+), 12 deletions(-)\n>>\n>> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n>> index d60d675..8091354 100644\n>> --- a/builtin/fast-export.c\n>> +++ b/builtin/fast-export.c\n>> @@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n> [snip]\n>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>         parse_commit(commit);\n>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>         if (!author)\n>> -               die (\"Could not find author in commit %s\",\n>> +               die(\"Could not find author in commit %s\",\n>>                      sha1_to_hex(commit->object.sha1));\n>\n> It looks like your simple replace didn't account for calls with\n> multiple lines.  Now the remaining lines don't line up.\n> :-)  There's several more places like this in the patch.\n\nGood eyes.\n\nMatching the coding-style to have no SP between function name and\nits argument list is just as important as matching the indentation\nstyle used in the project; trading one breakage with another does\nnot make much sense.\n"},{"id":"216835","messageId":"CAMP44s1HDp+ojGK0UhKHF=1iDu5_E9Z0VrK-JtMked1mtH_2gQ@mail.gmail.com","threadId":"33756","inReplyTo":"7v7gj77nt9.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-09T23:12:49Z","receivedAt":"2013-05-09T23:12:49Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 6:09 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> John Szakmeister <john@szakmeister.net> writes:\n>\n>> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n>> <felipe.contreras@gmail.com> wrote:\n>>> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n>>> ---\n>>>  builtin/fast-export.c | 24 ++++++++++++------------\n>>>  1 file changed, 12 insertions(+), 12 deletions(-)\n>>>\n>>> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n>>> index d60d675..8091354 100644\n>>> --- a/builtin/fast-export.c\n>>> +++ b/builtin/fast-export.c\n>>> @@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n>> [snip]\n>>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>>         parse_commit(commit);\n>>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>>         if (!author)\n>>> -               die (\"Could not find author in commit %s\",\n>>> +               die(\"Could not find author in commit %s\",\n>>>                      sha1_to_hex(commit->object.sha1));\n>>\n>> It looks like your simple replace didn't account for calls with\n>> multiple lines.  Now the remaining lines don't line up.\n>> :-)  There's several more places like this in the patch.\n>\n> Good eyes.\n>\n> Matching the coding-style to have no SP between function name and\n> its argument list is just as important as matching the indentation\n> style used in the project; trading one breakage with another does\n> not make much sense.\n\nWhere exactly in Documentation/CodingGuidelines is the \"indentation\nstyle\" used in the project specified that is being violated?\n\n-- \nFelipe Contreras\n"},{"id":"217546","messageId":"CAMP44s1jk=P=UhvwL_XzroqtneVDduZokYQb17qah9Z28HjT_g@mail.gmail.com","threadId":"33756","inReplyTo":"1368062218-22440-5-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH 4/4] fast-export: trivial cleanup","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T09:18:17Z","receivedAt":"2013-05-16T09:18:17Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Wed, May 8, 2013 at 8:16 PM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n> Cast the object to a commit, only to get the object back?\n\nI'm dropping this one.\n\n-- \nFelipe Contreras\n"},{"id":"217548","messageId":"CAMP44s3J8YpULYenDoYhyRJXSXL3b8-vpMbW2c4LcjK43Xidng@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s1HDp+ojGK0UhKHF=1iDu5_E9Z0VrK-JtMked1mtH_2gQ@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T09:23:41Z","receivedAt":"2013-05-16T09:23:41Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 9, 2013 at 6:12 PM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n> On Thu, May 9, 2013 at 6:09 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> John Szakmeister <john@szakmeister.net> writes:\n>>\n>>> On Wed, May 8, 2013 at 9:16 PM, Felipe Contreras\n>>> <felipe.contreras@gmail.com> wrote:\n>>>> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>\n>>>> ---\n>>>>  builtin/fast-export.c | 24 ++++++++++++------------\n>>>>  1 file changed, 12 insertions(+), 12 deletions(-)\n>>>>\n>>>> diff --git a/builtin/fast-export.c b/builtin/fast-export.c\n>>>> index d60d675..8091354 100644\n>>>> --- a/builtin/fast-export.c\n>>>> +++ b/builtin/fast-export.c\n>>>> @@ -135,7 +135,7 @@ static void export_blob(const unsigned char *sha1)\n>>> [snip]\n>>>> @@ -289,13 +289,13 @@ static void handle_commit(struct commit *commit, struct rev_info *rev)\n>>>>         parse_commit(commit);\n>>>>         author = strstr(commit->buffer, \"\\nauthor \");\n>>>>         if (!author)\n>>>> -               die (\"Could not find author in commit %s\",\n>>>> +               die(\"Could not find author in commit %s\",\n>>>>                      sha1_to_hex(commit->object.sha1));\n>>>\n>>> It looks like your simple replace didn't account for calls with\n>>> multiple lines.  Now the remaining lines don't line up.\n>>> :-)  There's several more places like this in the patch.\n>>\n>> Good eyes.\n>>\n>> Matching the coding-style to have no SP between function name and\n>> its argument list is just as important as matching the indentation\n>> style used in the project; trading one breakage with another does\n>> not make much sense.\n>\n> Where exactly in Documentation/CodingGuidelines is the \"indentation\n> style\" used in the project specified that is being violated?\n\nI find it extremely annoying that an obviously correct patch is not\nmerged because it's not conforming to a non-existing coding-style\nguideline that not even the Linux project follows. I've sent many\npatches where I change the alignment from cino=(0, to cino=(2s, and\nthe get applied because if it's not mentioned in\nDocumentation/CodingStyle, it cannot be used as a reason for\nrejection.\n\nI fixed the style so it conforms to Documentation/CodingGuidelines,\nand nowhere in there is the open parenthesis alignment mentioned, so\nusing that as a reason to reject this patch is a mistake in my\nopinion.\n\nIf you prefer the code to not follow Documentation/CodingGuidelines,\nso be it. I'm not going to work on this patch any more.\n\n-- \nFelipe Contreras\n"},{"id":"217617","messageId":"7vwqqy7v8g.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAMP44s3J8YpULYenDoYhyRJXSXL3b8-vpMbW2c4LcjK43Xidng@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T16:19:43Z","receivedAt":"2013-05-16T16:19:43Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> I find it extremely annoying that an obviously correct patch is not\n> merged...\n\nA patch that receives \"this breaks style\" comments and its author\nstubbornly refusing to fix what others pointed out, especially when\nthe patch itself _is_ about style, cannot be \"an obviously correct\"\npatch, can it?\n"},{"id":"217622","messageId":"CAMP44s17aD_ryeGWQazTcJ3nrhe6C9TLhKHhrUnys=Yj_ATa0A@mail.gmail.com","threadId":"33756","inReplyTo":"7vwqqy7v8g.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T16:32:47Z","receivedAt":"2013-05-16T16:32:47Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 11:19 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> I find it extremely annoying that an obviously correct patch is not\n>> merged...\n>\n> A patch that receives \"this breaks style\" comments\n\n*Invalid* \"this breaks style\" comments, because it doesn't break any\nstyle mentioned in Documentation/CodingGuidelines.\n\nIf somebody dreamt about another style not mentioned in\nDocumentation/CodingGuidelines after drinking too much coffe, that's\nirrelevant. The only relevant style is the defined style.\n\n> and its author\n> stubbornly refusing to fix what others pointed out, especially when\n> the patch itself _is_ about style, cannot be \"an obviously correct\"\n> patch, can it?\n\nThe patch makes the code follow Documentation/CodingGuidelines. Period.\n\n-- \nFelipe Contreras\n"},{"id":"217626","messageId":"7vzjvu6faq.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAMP44s17aD_ryeGWQazTcJ3nrhe6C9TLhKHhrUnys=Yj_ATa0A@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T16:49:17Z","receivedAt":"2013-05-16T16:49:17Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> On Thu, May 16, 2013 at 11:19 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>\n>>> I find it extremely annoying that an obviously correct patch is not\n>>> merged...\n>>\n>> A patch that receives \"this breaks style\" comments\n>\n> *Invalid* \"this breaks style\" comments, because it doesn't break any\n> style mentioned in Documentation/CodingGuidelines.\n\n    As for more concrete guidelines, just imitate the existing code\n    (this is a good guideline, no matter which project you are\n    contributing to). It is always preferable to match the _local_\n    convention\n\nThe document does not spell out all the details, but that does not\nmean you can ignore what others say and many find a sensible style\nrequest.\n"},{"id":"217628","messageId":"CAMP44s3c1vdCn43S=nSzfj=znZ9L_7jH9-+EOY0+SZNnSBTVbw@mail.gmail.com","threadId":"33756","inReplyTo":"7vzjvu6faq.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T17:02:19Z","receivedAt":"2013-05-16T17:02:19Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 11:49 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> On Thu, May 16, 2013 at 11:19 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>>\n>>>> I find it extremely annoying that an obviously correct patch is not\n>>>> merged...\n>>>\n>>> A patch that receives \"this breaks style\" comments\n>>\n>> *Invalid* \"this breaks style\" comments, because it doesn't break any\n>> style mentioned in Documentation/CodingGuidelines.\n>\n>     As for more concrete guidelines, just imitate the existing code\n>     (this is a good guideline, no matter which project you are\n>     contributing to). It is always preferable to match the _local_\n>     convention\n\nDo you want me to find places that follow the convention I used in my\npatch? Since the alignment is not even mentioned in the\nDocumentation/CodingGuidelines I I'll find them for sure.\n\nBut you are not interested in me finding that my style follows the\n_local_ convention. All you want is me to follow the style you want\nwithout going through the motions of updating the coding guidelines\nwhich would require comments from the community.\n\n> The document does not spell out all the details, but that does not\n> mean you can ignore what others say and many find a sensible style\n> request.\n\nYou don't know what's the point of a codding guideline? It's precisely\nto avoid these discussions.\n\nYou find it sensible, I don't. And when people disagree, that's what\nthe guideline is for. But since there's no guideline all we have is\nyour opinion vs. mine. And if you are going to play the \"I'm there\nauthority\" card, I'm not playing.\n\nLaw is law, guidelines are guidelines, and opinions are worthless.\n\n-- \nFelipe Contreras\n"},{"id":"217632","messageId":"7vfvxm6biv.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAMP44s3c1vdCn43S=nSzfj=znZ9L_7jH9-+EOY0+SZNnSBTVbw@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T18:10:48Z","receivedAt":"2013-05-16T18:10:48Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> You find it sensible, I don't. And when people disagree, that's what\n> the guideline is for. \n\nIf you really need it to be spelled out, I would say it would be\nsomething like this:\n\n    (defun linux-style ()\n      \"C mode with adjusted defaults for use with the Linux kernel.\"\n      (interactive)\n      (c-set-style \"K&R\")\n      (setq tab-width 8)\n      (setq indent-tabs-mode t)\n      (setq c-basic-offset 8))\n\n    (add-hook 'c-mode-hook 'linux-style)\n"},{"id":"217635","messageId":"CALkWK0=LuycRpuwBMjfiSk9qy2Y6VZsmmhNLpKdovKX4rZ=hTA@mail.gmail.com","threadId":"33756","inReplyTo":"7vfvxm6biv.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2013-05-16T18:34:39Z","receivedAt":"2013-05-16T18:34:39Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Junio C Hamano wrote:\n>     (defun linux-style ()\n>       \"C mode with adjusted defaults for use with the Linux kernel.\"\n>       (interactive)\n>       (c-set-style \"K&R\")\n\nThere's actually a \"linux\" style you can inherit from.\n\nMay I suggest looking at Documentation/CodingStyle in linux.git for this:\n\n(defun c-lineup-arglist-tabs-only (ignored)\n  \"Line up argument lists by tabs, not spaces\"\n  (let* ((anchor (c-langelem-pos c-syntactic-element))\n\t (column (c-langelem-2nd-pos c-syntactic-element))\n\t (offset (- (1+ column) anchor))\n\t (steps (floor offset c-basic-offset)))\n    (* (max steps 1)\n       c-basic-offset)))\n\n(add-hook 'c-mode-common-hook\n          (lambda ()\n            ;; Add kernel style\n            (c-add-style\n             \"linux-tabs-only\"\n             '(\"linux\" (c-offsets-alist\n                        (arglist-cont-nonempty\n                         c-lineup-gcc-asm-reg\n                         c-lineup-arglist-tabs-only))))))\n\n(add-hook 'c-mode-hook\n          (lambda ()\n            (let ((filename (buffer-file-name)))\n              ;; Enable kernel mode for the appropriate files\n              (when (and filename\n                         (string-match (expand-file-name \"~/src/linux-trees\")\n                                       filename))\n                (setq indent-tabs-mode t)\n                (c-set-style \"linux-tabs-only\")))))\n\nI use this for everything.\n"},{"id":"217665","messageId":"CAMP44s2QcFStPtE8cSbH7jWvUOUVRSgGNYNVoDN6exRdW6xNYA@mail.gmail.com","threadId":"33756","inReplyTo":"7vfvxm6biv.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T23:54:29Z","receivedAt":"2013-05-16T23:54:29Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 1:10 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> You find it sensible, I don't. And when people disagree, that's what\n>> the guideline is for.\n>\n> If you really need it to be spelled out, I would say it would be\n> something like this:\n>\n>     (defun linux-style ()\n>       \"C mode with adjusted defaults for use with the Linux kernel.\"\n>       (interactive)\n>       (c-set-style \"K&R\")\n>       (setq tab-width 8)\n>       (setq indent-tabs-mode t)\n>       (setq c-basic-offset 8))\n>\n>     (add-hook 'c-mode-hook 'linux-style)\n\n*You* are telling my that; it's *your* opinion and nothing else. It's\nnot the opinion of the *project*, and it's not the guideline of the\n*project*.\n\nWho says that maybe next week you won't change your opinion? Well,\nthat's what Documentation/CodingGuidelines is for.\n\nKeep rejecting patches arbitrarily on your whims, rather than\nestablished guidelines.\n\nThis is what Linux's checkpatch throws:\n\n---\n% ~/dev/linux/scripts/checkpatch.pl patch\nWARNING: line over 80 characters\n#108: FILE: builtin/fast-export.c:470:\n+\t\t\t\t\tdie(\"Can't find replacement commit for tag %s\\n\",\n\ntotal: 0 errors, 1 warnings, 92 lines checked\n\npatch has style problems, please review.\n\nIf any of these errors are false positives, please report\nthem to the maintainer, see CHECKPATCH in MAINTAINERS.\n--\n\nBut no, it's good enough for Linux, but it's not good enough for\nJunio? Why? Because he says so.\n\nSurely it cannot be an exercise in authoritarianism, if Junio says so,\nthe project does clearly have this guideline, it's just that is not\nexplicitly stated, and this style is violated, it should be in very\nfew places.\n\nSo lets run 'checkpatch --strict' for the whole project and lets see\nhow many places \"violate\" this style;\n\n---\nCHECK: Alignment should match open parenthesis\n#653: FILE: setup.c:653:\n+\t\tdie_errno(\"failed to stat '%*s%s%s'\",\n+\t\t\t\tprefix_len,\n\nCHECK: Alignment should match open parenthesis\n#248: FILE: commit.c:248:\n+\t\tmemmove(commit_graft + pos, commit_graft + pos + 1,\n+\t\t\t\tsizeof(struct commit_graft *)\n\nCHECK: Alignment should match open parenthesis\n#112: FILE: remote-testsvn.c:112:\n+\tif (!(msg = read_sha1_file(note_sha1, &type, &msglen)) ||\n+\t\t\t!msglen || type != OBJ_BLOB) {\n\nCHECK: Alignment should match open parenthesis\n#276: FILE: remote-testsvn.c:276:\n+\t\tif (!prefixcmp(line->buf, p->name) && (strlen(p->name) == line->len ||\n+\t\t\t\tline->buf[strlen(p->name)] == ' ')) {\n\nCHECK: Alignment should match open parenthesis\n#323: FILE: remote-testsvn.c:323:\n+\tstrbuf_addf(&marksfilename_sb, \"%s/info/fast-import/remote-svn/%s.marks\",\n+\t\tget_git_dir(), remote->name);\n\nCHECK: Alignment should match open parenthesis\n#61: FILE: levenshtein.c:61:\n+\t\t\tif (i > 0 && j > 0 && string1[i - 1] == string2[j] &&\n+\t\t\t\t\tstring1[i] == string2[j - 1] &&\n\nCHECK: Alignment should match open parenthesis\n#1105: FILE: config.c:1105:\n+\t\t\tif (strrchr(key, '.') - key == store.baselen &&\n+\t\t\t      !strncmp(key, store.key, store.baselen)) {\n\nCHECK: Alignment should match open parenthesis\n#1397: FILE: config.c:1397:\n+\t\t\tif (regcomp(store.value_regex, value_regex,\n+\t\t\t\t\tREG_EXTENDED)) {\n\nCHECK: Alignment should match open parenthesis\n#1434: FILE: config.c:1434:\n+\t\tif ((store.seen == 0 && value == NULL) ||\n+\t\t\t\t(store.seen > 1 && multi_replace == 0)) {\n\nCHECK: Alignment should match open parenthesis\n#214: FILE: read-cache.c:214:\n+\t\tif (ce->ce_uid != (unsigned int) st->st_uid ||\n+\t\t\tce->ce_gid != (unsigned int) st->st_gid)\n\nCHECK: Alignment should match open parenthesis\n#522: FILE: read-cache.c:522:\n+\tif (pos >= istate->cache_nr ||\n+\t\t\tcompare_name((ce = istate->cache[pos]), path, namelen))\n\nCHECK: Alignment should match open parenthesis\n#527: FILE: read-cache.c:527:\n+\tif (ce_stage(ce) == 1 && pos + 1 < istate->cache_nr &&\n+\t\t\tce_stage((ce = istate->cache[pos + 1])) == 2 &&\n\nCHECK: Alignment should match open parenthesis\n#834: FILE: imap-send.c:834:\n+\t\t\t\tfprintf(stderr, \"IMAP command '%s' returned response (%s) - %s\\n\",\n+\t\t\t\t\t memcmp(cmdp->cmd, \"LOGIN\", 5) ?\n\nCHECK: Alignment should match open parenthesis\n#28: FILE: test-svn-fe.c:28:\n+\tif (svndiff0_apply(&delta, (off_t) strtoumax(argv[4], NULL, 0),\n+\t\t\t\t\t&preimage_view, stdout))\n\nCHECK: Alignment should match open parenthesis\n#162: FILE: daemon.c:162:\n+\t\tstrbuf_expand(&expanded_path, interpolated_path,\n+\t\t\t\tstrbuf_expand_dict_cb, &dict);\n\nCHECK: Alignment should match open parenthesis\n#759: FILE: daemon.c:759:\n+\t\tinet_ntop(addr->sa_family, &sin_addr->sin_addr, addrbuf + 12,\n+\t\t    sizeof(addrbuf) - 12);\n\nCHECK: Alignment should match open parenthesis\n#761: FILE: daemon.c:761:\n+\t\tsnprintf(portbuf, sizeof(portbuf), \"REMOTE_PORT=%d\",\n+\t\t    ntohs(sin_addr->sin_port));\n\nCHECK: Alignment should match open parenthesis\n#769: FILE: daemon.c:769:\n+\t\tinet_ntop(AF_INET6, &sin6_addr->sin6_addr, buf,\n+\t\t    sizeof(addrbuf) - 13);\n\nCHECK: Alignment should match open parenthesis\n#773: FILE: daemon.c:773:\n+\t\tsnprintf(portbuf, sizeof(portbuf), \"REMOTE_PORT=%d\",\n+\t\t    ntohs(sin6_addr->sin6_port));\n\nCHECK: Alignment should match open parenthesis\n#1017: FILE: daemon.c:1017:\n+\t\t\t\tlogerror(\"Poll failed, resuming: %s\",\n+\t\t\t\t      strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#1092: FILE: daemon.c:1092:\n+\tif (cred && (initgroups(cred->pass->pw_name, cred->gid) ||\n+\t    setgid (cred->gid) || setuid(cred->pass->pw_uid)))\n\nCHECK: Alignment should match open parenthesis\n#103: FILE: vcs-svn/fast_export.c:103:\n+\t\tsnprintf(gitsvnline, MAX_GITSVN_LINE_LEN,\n+\t\t\t\t\"\\n\\ngit-svn-id: %s@%\"PRIu32\" %s\\n\",\n\nCHECK: Alignment should match open parenthesis\n#111: FILE: vcs-svn/fast_export.c:111:\n+\tprintf(\"committer %s <%s@%s> %ld +0000\\n\",\n+\t\t   *author ? author : \"nobody\",\n\nCHECK: Alignment should match open parenthesis\n#115: FILE: vcs-svn/fast_export.c:115:\n+\tprintf(\"data %\"PRIuMAX\"\\n\",\n+\t\t(uintmax_t) (log->len + strlen(gitsvnline)));\n\nCHECK: Alignment should match open parenthesis\n#300: FILE: vcs-svn/svndump.c:300:\n+\tfast_export_blob_delta(node_ctx.type, old_mode, old_data,\n+\t\t\t\tnode_ctx.text_length, &input);\n\nCHECK: Alignment should match open parenthesis\n#308: FILE: vcs-svn/svndump.c:308:\n+\tfast_export_begin_commit(rev_ctx.revision, rev_ctx.author.buf,\n+\t\t&rev_ctx.log, dump_ctx.uuid.buf, dump_ctx.url.buf,\n\nCHECK: Alignment should match open parenthesis\n#318: FILE: vcs-svn/svndump.c:318:\n+\t\tfast_export_begin_note(rev_ctx.revision, \"remote-svn\",\n+\t\t\t\t\"Note created by remote-svn.\", rev_ctx.timestamp, note_ref);\n\nCHECK: Alignment should match open parenthesis\n#238: FILE: wrapper.c:238:\n+\t\tdie_errno(\"Unable to create temporary file '%s'\",\n+\t\t\tnonrelative_template);\n\nCHECK: Alignment should match open parenthesis\n#373: FILE: wrapper.c:373:\n+\t\tdie_errno(\"Unable to create temporary file '%s'\",\n+\t\t\tnonrelative_template);\n\nCHECK: Alignment should match open parenthesis\n#239: FILE: ws.c:239:\n+\t\t\tfwrite(line + written,\n+\t\t\t    trailing_whitespace - written, 1, stream);\n\nCHECK: Alignment should match open parenthesis\n#247: FILE: ws.c:247:\n+\t\t\tfwrite(line + trailing_whitespace,\n+\t\t\t    len - trailing_whitespace, 1, stream);\n\nCHECK: Alignment should match open parenthesis\n#14: FILE: contrib/svn-fe/svn-fe.c:14:\n+\tsvndump_read((argc > 1) ? argv[1] : NULL, \"refs/heads/master\",\n+\t\t\t\"refs/notes/svn/revs\");\n\nCHECK: Alignment should match open parenthesis\n#167: FILE: contrib/credential/wincred/git-credential-wincred.c:167:\n+\t\t\twrite_item(\"username\", creds[i]->UserName,\n+\t\t\t\twcslen(creds[i]->UserName));\n\nCHECK: Alignment should match open parenthesis\n#169: FILE: contrib/credential/wincred/git-credential-wincred.c:169:\n+\t\t\twrite_item(\"password\",\n+\t\t\t\t(LPCWSTR)creds[i]->CredentialBlob,\n\nCHECK: Alignment should match open parenthesis\n#680: FILE: upload-pack.c:680:\n+\t\t\t\tpacket_write(1, \"shallow %s\",\n+\t\t\t\t\t\tsha1_to_hex(object->sha1));\n\nCHECK: Alignment should match open parenthesis\n#692: FILE: upload-pack.c:692:\n+\t\t\t\tpacket_write(1, \"unshallow %s\",\n+\t\t\t\t\tsha1_to_hex(object->sha1));\n\nCHECK: Alignment should match open parenthesis\n#702: FILE: upload-pack.c:702:\n+\t\t\t\t\tadd_object_array(&parents->item->object,\n+\t\t\t\t\t\t\tNULL, &want_obj);\n\nCHECK: Alignment should match open parenthesis\n#556: FILE: transport-helper.c:556:\n+\t\tdie(\"Unknown response to connect: %s\",\n+\t\t\tcmdbuf.buf);\n\nCHECK: Alignment should match open parenthesis\n#1038: FILE: transport-helper.c:1038:\n+\tif (bytes < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\terrno != EINTR) {\n\nCHECK: Alignment should match open parenthesis\n#1043: FILE: transport-helper.c:1043:\n+\t\ttransfer_debug(\"%s EOF (with %i bytes in buffer)\",\n+\t\t\tt->src_name, t->bufuse);\n\nCHECK: Alignment should match open parenthesis\n#1048: FILE: transport-helper.c:1048:\n+\t\ttransfer_debug(\"Read %i bytes from %s (buffer now at %i)\",\n+\t\t\t(int)bytes, t->src_name, (int)t->bufuse);\n\nCHECK: Alignment should match open parenthesis\n#1066: FILE: transport-helper.c:1066:\n+\tif (bytes < 0 && errno != EWOULDBLOCK && errno != EAGAIN &&\n+\t\terrno != EINTR) {\n\nCHECK: Alignment should match open parenthesis\n#1074: FILE: transport-helper.c:1074:\n+\t\ttransfer_debug(\"Wrote %i bytes to %s (buffer now at %i)\",\n+\t\t\t(int)bytes, t->dest_name, (int)t->bufuse);\n\nCHECK: Alignment should match open parenthesis\n#142: FILE: send-pack.c:142:\n+\t\t\twarning(\"remote reported status on unknown ref: %s\",\n+\t\t\t\t\trefname);\n\nCHECK: Alignment should match open parenthesis\n#147: FILE: send-pack.c:147:\n+\t\t\twarning(\"remote reported status on unexpected ref: %s\",\n+\t\t\t\t\trefname);\n\nCHECK: Alignment should match open parenthesis\n#87: FILE: http-fetch.c:87:\n+\t\tfprintf(stderr,\n+\"Some loose object were found to be corrupt, but they might be just\\n\"\n\nCHECK: Alignment should match open parenthesis\n#335: FILE: sha1_file.c:335:\n+\t\terror(\"%s: ignoring alternate object stores, nesting too deep.\",\n+\t\t\t\trelative_base);\n\nCHECK: Alignment should match open parenthesis\n#347: FILE: sha1_file.c:347:\n+\t\t\terror(\"%s: ignoring relative alternate object store %s\",\n+\t\t\t\t\trelative_base, entry);\n\nCHECK: Alignment should match open parenthesis\n#900: FILE: sha1_file.c:900:\n+\t\t\t\tdie(\"packfile %s cannot be mapped: %s\",\n+\t\t\t\t\tp->pack_name,\n\nCHECK: Alignment should match open parenthesis\n#778: FILE: combine-diff.c:778:\n+\t\t\t\tprintf(\"%s%s %s%s\", c_reset,\n+\t\t\t\t\t\t    c_plain, c_reset,\n\nCHECK: Alignment should match open parenthesis\n#444: FILE: pretty.c:444:\n+\t\t\tstrbuf_add_wrapped_bytes(sb, quoted.buf, quoted.len,\n+\t\t\t\t\t\t\t-6, 1, max_length);\n\nCHECK: Alignment should match open parenthesis\n#581: FILE: pretty.c:581:\n+\t\tstrbuf_splice(&tmp, start + strlen(\"encoding \"),\n+\t\t\t\t\t  len - strlen(\"encoding \\n\"),\n\nCHECK: Alignment should match open parenthesis\n#1596: FILE: pretty.c:1596:\n+\t\t\tadd_rfc2047(sb, title.buf, title.len,\n+\t\t\t\t\t\tencoding, RFC2047_SUBJECT);\n\nCHECK: Alignment should match open parenthesis\n#1599: FILE: pretty.c:1599:\n+\t\t\tstrbuf_add_wrapped_bytes(sb, title.buf, title.len,\n+\t\t\t\t\t -last_line_length(sb), 1, max_length);\n\nCHECK: Alignment should match open parenthesis\n#202: FILE: rerere.c:202:\n+\t\t\t\tgit_SHA1_Update(&ctx, one.buf ? one.buf : \"\",\n+\t\t\t\t\t    one.len + 1);\n\nCHECK: Alignment should match open parenthesis\n#204: FILE: rerere.c:204:\n+\t\t\t\tgit_SHA1_Update(&ctx, two.buf ? two.buf : \"\",\n+\t\t\t\t\t    two.len + 1);\n\nCHECK: Alignment should match open parenthesis\n#443: FILE: rerere.c:443:\n+\tif (read_mmfile(&cur, rerere_path(name, \"thisimage\")) ||\n+\t\t\tread_mmfile(&base, rerere_path(name, \"preimage\")) ||\n\nCHECK: Alignment should match open parenthesis\n#454: FILE: rerere.c:454:\n+\t\t\twarning(\"failed utime() on %s: %s\",\n+\t\t\t\t\trerere_path(name, \"postimage\"),\n\nCHECK: Alignment should match open parenthesis\n#97: FILE: shallow.c:97:\n+\t\t\t\t\tadd_object_array(&p->item->object,\n+\t\t\t\t\t\t\tNULL, &stack);\n\nCHECK: Alignment should match open parenthesis\n#250: FILE: diff.c:250:\n+\tif (!strcmp(var, \"diff.suppressblankempty\") ||\n+\t\t\t/* for backwards compatibility */\n\nCHECK: Alignment should match open parenthesis\n#826: FILE: diff.c:826:\n+\tif ((diff_words->last_minus == 0 &&\n+\t\tdiff_words->current_plus == diff_words->plus.text.ptr) ||\n\nCHECK: Alignment should match open parenthesis\n#845: FILE: diff.c:845:\n+\tif (line[0] != '@' || parse_hunk_header(line, len,\n+\t\t\t&minus_first, &minus_len, &plus_first, &plus_len))\n\nCHECK: Alignment should match open parenthesis\n#871: FILE: diff.c:871:\n+\t\tfn_out_diff_words_write_helper(diff_words->opt->file,\n+\t\t\t\t&style->ctx, style->newline,\n\nCHECK: Alignment should match open parenthesis\n#879: FILE: diff.c:879:\n+\t\tfn_out_diff_words_write_helper(diff_words->opt->file,\n+\t\t\t\t&style->old, style->newline,\n\nCHECK: Alignment should match open parenthesis\n#885: FILE: diff.c:885:\n+\t\tfn_out_diff_words_write_helper(diff_words->opt->file,\n+\t\t\t\t&style->new, style->newline,\n\nCHECK: Alignment should match open parenthesis\n#949: FILE: diff.c:949:\n+\t\tALLOC_GROW(buffer->orig, buffer->orig_nr + 1,\n+\t\t\t\tbuffer->orig_alloc);\n\nCHECK: Alignment should match open parenthesis\n#982: FILE: diff.c:982:\n+\t\tfn_out_diff_words_write_helper(diff_words->opt->file,\n+\t\t\t&style->old, style->newline,\n\nCHECK: Alignment should match open parenthesis\n#1008: FILE: diff.c:1008:\n+\t\tfn_out_diff_words_write_helper(diff_words->opt->file,\n+\t\t\t&style->ctx, style->newline,\n\nCHECK: Alignment should match open parenthesis\n#2253: FILE: diff.c:2253:\n+\tif (DIFF_OPT_TST(o, SUBMODULE_LOG) &&\n+\t\t\t(!one->mode || S_ISGITLINK(one->mode)) &&\n\nCHECK: Alignment should match open parenthesis\n#2258: FILE: diff.c:2258:\n+\t\tshow_submodule_summary(o->file, one ? one->path : two->path,\n+\t\t\t\tline_prefix,\n\nCHECK: Alignment should match open parenthesis\n#2321: FILE: diff.c:2321:\n+\t\t\temit_rewrite_diff(name_a, name_b, one, two,\n+\t\t\t\t\t\ttextconv_one, textconv_two, o);\n\nCHECK: Alignment should match open parenthesis\n#2801: FILE: diff.c:2801:\n+\tif (convert_to_working_tree(path,\n+\t\t\t(const char *)blob, (size_t)size, &buf)) {\n\nCHECK: Alignment should match open parenthesis\n#3496: FILE: diff.c:3496:\n+\t\tdie(_(\"Failed to parse --submodule option parameter: '%s'\"),\n+\t\t\tvalue);\n\nCHECK: Alignment should match open parenthesis\n#4272: FILE: diff.c:4272:\n+\t\tif (fill_mmfile(&mf1, p->one) < 0 ||\n+\t\t\t\tfill_mmfile(&mf2, p->two) < 0)\n\nCHECK: Alignment should match open parenthesis\n#196: FILE: remote-curl.c:196:\n+\tif ((!prefixcmp(url, \"http://\") || !prefixcmp(url, \"https://\")) &&\n+\t     git_env_bool(\"GIT_SMART_HTTP\", 1)) {\n\nCHECK: Alignment should match open parenthesis\n#1420: FILE: remote.c:1420:\n+\t\tif (!ref->deletion &&\n+\t\t\t!hashcmp(ref->old_sha1, ref->new_sha1)) {\n\nCHECK: Alignment should match open parenthesis\n#1781: FILE: remote.c:1781:\n+\t\tstrbuf_addf(sb,\n+\t\t\tQ_(\"Your branch is ahead of '%s' by %d commit.\\n\",\n\nCHECK: Alignment should match open parenthesis\n#1787: FILE: remote.c:1787:\n+\t\t\tstrbuf_addf(sb,\n+\t\t\t\t_(\"  (use \\\"git push\\\" to publish your local commits)\\n\"));\n\nCHECK: Alignment should match open parenthesis\n#1790: FILE: remote.c:1790:\n+\t\tstrbuf_addf(sb,\n+\t\t\tQ_(\"Your branch is behind '%s' by %d commit, \"\n\nCHECK: Alignment should match open parenthesis\n#1798: FILE: remote.c:1798:\n+\t\t\tstrbuf_addf(sb,\n+\t\t\t\t_(\"  (use \\\"git pull\\\" to update your local branch)\\n\"));\n\nCHECK: Alignment should match open parenthesis\n#1801: FILE: remote.c:1801:\n+\t\tstrbuf_addf(sb,\n+\t\t\tQ_(\"Your branch and '%s' have diverged,\\n\"\n\nCHECK: Alignment should match open parenthesis\n#1811: FILE: remote.c:1811:\n+\t\t\tstrbuf_addf(sb,\n+\t\t\t\t_(\"  (use \\\"git pull\\\" to merge the remote branch into yours)\\n\"));\n\nCHECK: Alignment should match open parenthesis\n#395: FILE: http-backend.c:395:\n+\t\tstrbuf_addf(&buf, \"application/x-git-%s-advertisement\",\n+\t\t\tsvc->name);\n\nCHECK: Alignment should match open parenthesis\n#479: FILE: http-backend.c:479:\n+\t\tformat_write(1,\n+\t\t\t\"Expected POST with Content-Type '%s',\"\n\nCHECK: Alignment should match open parenthesis\n#92: FILE: test-chmtime.c:92:\n+\t\tif (!(sb.st_mode & S_IWUSR) &&\n+\t\t\t\tchmod(argv[i], sb.st_mode | S_IWUSR)) {\n\nCHECK: Alignment should match open parenthesis\n#806: FILE: wt-status.c:806:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (fix conflicts and run \\\"git commit\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#809: FILE: wt-status.c:809:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"All conflicts fixed but you are still merging.\"));\n\nCHECK: Alignment should match open parenthesis\n#812: FILE: wt-status.c:812:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (use \\\"git commit\\\" to conclude merge)\"));\n\nCHECK: Alignment should match open parenthesis\n#822: FILE: wt-status.c:822:\n+\tstatus_printf_ln(s, color,\n+\t\t_(\"You are in the middle of an am session.\"));\n\nCHECK: Alignment should match open parenthesis\n#825: FILE: wt-status.c:825:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"The current patch is empty.\"));\n\nCHECK: Alignment should match open parenthesis\n#829: FILE: wt-status.c:829:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (fix conflicts and then run \\\"git am --resolved\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#831: FILE: wt-status.c:831:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"  (use \\\"git am --skip\\\" to skip this patch)\"));\n\nCHECK: Alignment should match open parenthesis\n#833: FILE: wt-status.c:833:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"  (use \\\"git am --abort\\\" to restore the original branch)\"));\n\nCHECK: Alignment should match open parenthesis\n#901: FILE: wt-status.c:901:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (fix conflicts and then run \\\"git rebase --continue\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#903: FILE: wt-status.c:903:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (use \\\"git rebase --skip\\\" to skip this patch)\"));\n\nCHECK: Alignment should match open parenthesis\n#905: FILE: wt-status.c:905:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (use \\\"git rebase --abort\\\" to check out the original branch)\"));\n\nCHECK: Alignment should match open parenthesis\n#918: FILE: wt-status.c:918:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (all conflicts fixed: run \\\"git rebase --continue\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#930: FILE: wt-status.c:930:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (Once your working directory is clean, run \\\"git rebase\n--continue\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#942: FILE: wt-status.c:942:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (use \\\"git commit --amend\\\" to amend the current commit)\"));\n\nCHECK: Alignment should match open parenthesis\n#944: FILE: wt-status.c:944:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (use \\\"git rebase --continue\\\" once you are satisfied with\nyour changes)\"));\n\nCHECK: Alignment should match open parenthesis\n#958: FILE: wt-status.c:958:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (fix conflicts and run \\\"git commit\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#961: FILE: wt-status.c:961:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (all conflicts fixed: run \\\"git commit\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#975: FILE: wt-status.c:975:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (fix conflicts and run \\\"git revert --continue\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#978: FILE: wt-status.c:978:\n+\t\t\tstatus_printf_ln(s, color,\n+\t\t\t\t_(\"  (all conflicts fixed: run \\\"git revert --continue\\\")\"));\n\nCHECK: Alignment should match open parenthesis\n#980: FILE: wt-status.c:980:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"  (use \\\"git revert --abort\\\" to cancel the revert operation)\"));\n\nCHECK: Alignment should match open parenthesis\n#998: FILE: wt-status.c:998:\n+\t\tstatus_printf_ln(s, color,\n+\t\t\t_(\"  (use \\\"git bisect reset\\\" to get back to the original branch)\"));\n\nCHECK: Alignment should match open parenthesis\n#1233: FILE: wt-status.c:1233:\n+\t\tstatus_printf_ln(s, GIT_COLOR_NORMAL, _(\"Untracked files not listed%s\"),\n+\t\t\tadvice_status_hints\n\nCHECK: Alignment should match open parenthesis\n#98: FILE: xdiff/xhistogram.c:98:\n+\t\txdl_recmatch(r1->ptr, r1->size, r2->ptr, r2->size,\n+\t\t\t    xpp->flags);\n\nCHECK: Alignment should match open parenthesis\n#75: FILE: xdiff/xemit.c:75:\n+\tif (len > 0 &&\n+\t\t\t(isalpha((unsigned char)*rec) || /* identifier? */\n\nCHECK: Alignment should match open parenthesis\n#94: FILE: xdiff/xpatience.c:94:\n+\t\tif (map->entries[index].hash != record->ha ||\n+\t\t\t\t!xdl_recmatch(record->ptr, record->size,\n\nCHECK: Alignment should match open parenthesis\n#266: FILE: xdiff/xpatience.c:266:\n+\t\t\tif (patience_diff(map->file1, map->file2,\n+\t\t\t\t\tmap->xpp, map->env,\n\nCHECK: Alignment should match open parenthesis\n#324: FILE: xdiff/xpatience.c:324:\n+\tif (fill_hashmap(file1, file2, xpp, env, &map,\n+\t\t\tline1, count1, line2, count2))\n\nCHECK: Alignment should match open parenthesis\n#348: FILE: xdiff/xmerge.c:348:\n+\t\tif (line_contains_alnum(xe->xdf2.recs[i]->ptr,\n+\t\t\t\txe->xdf2.recs[i]->size))\n\nCHECK: Alignment should match open parenthesis\n#468: FILE: xdiff/xmerge.c:468:\n+\t\tif (level == XDL_MERGE_MINIMAL || xscr1->i1 != xscr2->i1 ||\n+\t\t\t\txscr1->chg1 != xscr2->chg1 ||\n\nCHECK: Alignment should match open parenthesis\n#471: FILE: xdiff/xmerge.c:471:\n+\t\t\t\txdl_merge_cmp_lines(xe1, xscr1->i2,\n+\t\t\t\t\txe2, xscr2->i2,\n\nCHECK: Alignment should match open parenthesis\n#583: FILE: xdiff/xmerge.c:583:\n+\tif (xdl_do_diff(orig, mf1, xpp, &xe1) < 0 ||\n+\t\t\txdl_do_diff(orig, mf2, xpp, &xe2) < 0) {\n\nCHECK: Alignment should match open parenthesis\n#121: FILE: xdiff/xprepare.c:121:\n+\t\tif (rcrec->ha == rec->ha &&\n+\t\t\t\txdl_recmatch(rcrec->line, rcrec->size,\n\nCHECK: Alignment should match open parenthesis\n#122: FILE: xdiff/xprepare.c:122:\n+\t\t\t\txdl_recmatch(rcrec->line, rcrec->size,\n+\t\t\t\t\trec->ptr, rec->size, cf->flags))\n\nCHECK: Alignment should match open parenthesis\n#325: FILE: archive.c:325:\n+\t\tOPT_STRING(0, \"prefix\", &base, N_(\"prefix\"),\n+\t\t\tN_(\"prepend prefix to each pathname in the archive\")),\n\nCHECK: Alignment should match open parenthesis\n#327: FILE: archive.c:327:\n+\t\tOPT_STRING('o', \"output\", &output, N_(\"file\"),\n+\t\t\tN_(\"write the archive to this file\")),\n\nCHECK: Alignment should match open parenthesis\n#329: FILE: archive.c:329:\n+\t\tOPT_BOOL(0, \"worktree-attributes\", &worktree_attributes,\n+\t\t\tN_(\"read .gitattributes in working directory\")),\n\nCHECK: Alignment should match open parenthesis\n#343: FILE: archive.c:343:\n+\t\tOPT_BOOL('l', \"list\", &list,\n+\t\t\tN_(\"list supported archive formats\")),\n\nCHECK: Alignment should match open parenthesis\n#346: FILE: archive.c:346:\n+\t\tOPT_STRING(0, \"remote\", &remote, N_(\"repo\"),\n+\t\t\tN_(\"retrieve the archive from remote repository <repo>\")),\n\nCHECK: Alignment should match open parenthesis\n#348: FILE: archive.c:348:\n+\t\tOPT_STRING(0, \"exec\", &exec, N_(\"command\"),\n+\t\t\tN_(\"path to the remote git-upload-archive command\")),\n\nCHECK: Alignment should match open parenthesis\n#389: FILE: archive.c:389:\n+\t\t\tdie(\"Argument not supported for format '%s': -%d\",\n+\t\t\t\t\tformat, compression_level);\n\nCHECK: Alignment should match open parenthesis\n#160: FILE: diffcore-delta.c:160:\n+\tqsort(hash->data,\n+\t\t1ul << hash->alloc_log2,\n\nCHECK: Alignment should match open parenthesis\n#109: FILE: merge.c:109:\n+\tif (write_cache(fd, active_cache, active_nr) ||\n+\t\tcommit_locked_index(lock_file))\n\nCHECK: Alignment should match open parenthesis\n#377: FILE: help.c:377:\n+\t\t\tfprintf_ln(stderr, _(\"in %0.1f seconds automatically...\"),\n+\t\t\t\t(float)autocorrect/10.0);\n\nCHECK: Alignment should match open parenthesis\n#298: FILE: diff-lib.c:298:\n+\tif (get_stat_data(new, &sha1, &mode, cached, match_missing,\n+\t    &dirty_submodule, &revs->diffopt) < 0)\n\nCHECK: Alignment should match open parenthesis\n#961: FILE: fast-import.c:961:\n+\t\tfixup_pack_header_footer(pack_data->pack_fd, pack_data->sha1,\n+\t\t\t\t    pack_data->pack_name, object_count,\n\nCHECK: Alignment should match open parenthesis\n#1413: FILE: fast-import.c:1413:\n+\t\tstrbuf_addf(b, \"%o %s%c\",\n+\t\t\t(unsigned int)(e->versions[v].mode & ~NO_DELTA),\n\nCHECK: Alignment should match open parenthesis\n#1742: FILE: fast-import.c:1742:\n+\t\t\t\tdump_marks_helper(f, base + (k << m->shift),\n+\t\t\t\t\tm->data.sets[k]);\n\nCHECK: Alignment should match open parenthesis\n#1950: FILE: fast-import.c:1950:\n+\t\t\t\tdie(\"EOF in data (%lu bytes remaining)\",\n+\t\t\t\t\t(unsigned long)(length - n));\n\nCHECK: Alignment should match open parenthesis\n#2185: FILE: fast-import.c:2185:\n+\t\t\ttree_content_set(orig_root, realpath,\n+\t\t\t\tleaf.versions[1].sha1,\n\nCHECK: Alignment should match open parenthesis\n#2324: FILE: fast-import.c:2324:\n+\t\t\tdie(\"Git links cannot be specified 'inline': %s\",\n+\t\t\t\tcommand_buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#2328: FILE: fast-import.c:2328:\n+\t\t\t\tdie(\"Not a commit (actually a %s): %s\",\n+\t\t\t\t\ttypename(oe->type), command_buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#2337: FILE: fast-import.c:2337:\n+\t\t\tdie(\"Directories cannot be specified 'inline': %s\",\n+\t\t\t\tcommand_buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#2351: FILE: fast-import.c:2351:\n+\t\t\tdie(\"%s not found: %s\",\n+\t\t\t\t\tS_ISDIR(mode) ?  \"Tree\" : \"Blob\",\n\nCHECK: Alignment should match open parenthesis\n#2355: FILE: fast-import.c:2355:\n+\t\t\tdie(\"Not a %s (actually a %s): %s\",\n+\t\t\t\ttypename(expected), typename(type),\n\nCHECK: Alignment should match open parenthesis\n#2423: FILE: fast-import.c:2423:\n+\t\ttree_content_replace(&b->branch_tree,\n+\t\t\tleaf.versions[1].sha1,\n\nCHECK: Alignment should match open parenthesis\n#2429: FILE: fast-import.c:2429:\n+\ttree_content_set(&b->branch_tree, d,\n+\t\tleaf.versions[1].sha1,\n\nCHECK: Alignment should match open parenthesis\n#2513: FILE: fast-import.c:2513:\n+\t\t\tdie(\"Not a blob (actually a %s): %s\",\n+\t\t\t\ttypename(oe->type), command_buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#2732: FILE: fast-import.c:2732:\n+\tstrbuf_addf(&new_data, \"tree %s\\n\",\n+\t\tsha1_to_hex(b->branch_tree.versions[1].sha1));\n\nCHECK: Alignment should match open parenthesis\n#2742: FILE: fast-import.c:2742:\n+\tstrbuf_addf(&new_data,\n+\t\t\"author %s\\n\"\n\nCHECK: Alignment should match open parenthesis\n#2901: FILE: fast-import.c:2901:\n+\tstrbuf_addf(&line, \"%s %s %lu\\n\", sha1_to_hex(sha1),\n+\t\t\t\t\t\ttypename(type), size);\n\nCHECK: Alignment should match open parenthesis\n#3031: FILE: fast-import.c:3031:\n+\t\tstrbuf_addf(&line, \"%06o %s %s\\t\",\n+\t\t\t\tmode & ~NO_DELTA, type, sha1_to_hex(sha1));\n\nCHECK: Alignment should match open parenthesis\n#3236: FILE: fast-import.c:3236:\n+\t\toption_import_marks(feature + strlen(\"import-marks-if-exists=\"),\n+\t\t\t\t\tfrom_stream, 1);\n\nCHECK: Alignment should match open parenthesis\n#169: FILE: strbuf.c:169:\n+\tmemmove(sb->buf + pos + dlen,\n+\t\t\tsb->buf + pos + len,\n\nCHECK: Alignment should match open parenthesis\n#310: FILE: fetch-pack.c:310:\n+\t\tif (((o = lookup_object(remote)) != NULL) &&\n+\t\t\t\t(o->flags & COMPLETE)) {\n\nCHECK: Alignment should match open parenthesis\n#413: FILE: fetch-pack.c:413:\n+\t\t\t\t\tfprintf(stderr, \"got ack %d %s\\n\", ack,\n+\t\t\t\t\t\t\tsha1_to_hex(result_sha1));\n\nCHECK: Alignment should match open parenthesis\n#439: FILE: path.c:439:\n+\tif (((old_mode ^ new_mode) & ~S_IFMT) &&\n+\t\t\tchmod(path, (new_mode & ~S_IFMT)) < 0)\n\nCHECK: Alignment should match open parenthesis\n#318: FILE: merge-recursive.c:318:\n+\tget_tree_entry(o->object.sha1, path,\n+\t\t\te->stages[1].sha, &e->stages[1].mode);\n\nCHECK: Alignment should match open parenthesis\n#320: FILE: merge-recursive.c:320:\n+\tget_tree_entry(a->object.sha1, path,\n+\t\t\te->stages[2].sha, &e->stages[2].mode);\n\nCHECK: Alignment should match open parenthesis\n#322: FILE: merge-recursive.c:322:\n+\tget_tree_entry(b->object.sha1, path,\n+\t\t\te->stages[3].sha, &e->stages[3].mode);\n\nCHECK: Alignment should match open parenthesis\n#1635: FILE: merge-recursive.c:1635:\n+\t\toutput(o, 1, _(\"CONFLICT (%s): Merge conflict in %s\"),\n+\t\t\t\treason, path);\n\nCHECK: Alignment should match open parenthesis\n#2007: FILE: merge-recursive.c:2007:\n+\tif (active_cache_changed &&\n+\t\t\t(write_cache(index_fd, active_cache, active_nr) ||\n\nCHECK: Alignment should match open parenthesis\n#370: FILE: sequencer.c:370:\n+\t\tif (cache_tree_update(active_cache_tree, active_cache,\n+\t\t\t\t  active_nr, 0))\n\nCHECK: Alignment should match open parenthesis\n#685: FILE: sequencer.c:685:\n+\t\tstrbuf_addf(buf, \"%s %s %.*s\\n\", action_str, sha1_abbrev,\n+\t\t\tsubject_len, subject);\n\nCHECK: Alignment should match open parenthesis\n#917: FILE: sequencer.c:917:\n+\t\terror(_(\"stored pre-cherry-pick HEAD file '%s' is corrupt\"),\n+\t\t\tfilename);\n\nCHECK: Alignment should match open parenthesis\n#990: FILE: sequencer.c:990:\n+\t\tassert(!(opts->signoff || opts->no_commit ||\n+\t\t\t\topts->record_origin || opts->edit));\n\nCHECK: Alignment should match open parenthesis\n#1135: FILE: sequencer.c:1135:\n+\tstrbuf_addstr(&sob, fmt_name(getenv(\"GIT_COMMITTER_NAME\"),\n+\t\t\t\tgetenv(\"GIT_COMMITTER_EMAIL\")));\n\nCHECK: Alignment should match open parenthesis\n#1182: FILE: sequencer.c:1182:\n+\t\t\tstrbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,\n+\t\t\t\tappend_newlines, strlen(append_newlines));\n\nCHECK: Alignment should match open parenthesis\n#1187: FILE: sequencer.c:1187:\n+\t\tstrbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,\n+\t\t\t\tsob.buf, sob.len);\n\nCHECK: Alignment should match open parenthesis\n#76: FILE: compat/win32/syslog.c:76:\n+\tReportEventA(ms_eventlog, logtype, 0, 0, NULL, 1, 0,\n+\t    (const char **)&str, NULL);\n\nCHECK: Alignment should match open parenthesis\n#89: FILE: compat/winansi.c:89:\n+\tFillConsoleOutputCharacterA(console, ' ',\n+\t\tsbi.dwSize.X - sbi.dwCursorPosition.X, sbi.dwCursorPosition,\n\nCHECK: Alignment should match open parenthesis\n#453: FILE: compat/nedmalloc/nedmalloc.c:453:\n+\t\tprintf(\"*** threadcache=%u, mallocs=%u (%f), free=%u (%f),\nfreeInCache=%u\\n\", (unsigned int) tc->threadid, tc->mallocs,\n+\t\t\t(float) tc->successes/tc->mallocs, tc->frees, (float)\ntc->successes/tc->frees, (unsigned int) tc->freeInCache);\n\nCHECK: Alignment should match open parenthesis\n#701: FILE: compat/nedmalloc/nedmalloc.c:701:\n+\t\tprintf(\"Threadcache utilisation: %lf%% in cache with %lf%% lost to\nother threads\\n\",\n+\t\t\t100.0*tc->successes/tc->mallocs, 100.0*((double)\ntc->mallocs-tc->frees)/tc->mallocs);\n\nCHECK: Alignment should match open parenthesis\n#947: FILE: compat/nedmalloc/nedmalloc.c:947:\n+\tGETMSPACE(m, p, tc, mymspace, 0,\n+\t      ret=mspace_independent_comalloc(m, elems, adjustedsizes, chunks));\n\nCHECK: Alignment should match open parenthesis\n#432: FILE: compat/mingw.c:432:\n+\t\t\t\tif ((findbuf.dwFileAttributes & FILE_ATTRIBUTE_REPARSE_POINT) &&\n+\t\t\t\t\t\t(findbuf.dwReserved0 == IO_REPARSE_TAG_SYMLINK)) {\n\nCHECK: Alignment should match open parenthesis\n#1342: FILE: compat/mingw.c:1342:\n+\t\tdie(\"unable to initialize winsock subsystem, error %d\",\n+\t\t\tWSAGetLastError());\n\nCHECK: Alignment should match open parenthesis\n#1538: FILE: compat/mingw.c:1538:\n+\tif (gle == ERROR_ACCESS_DENIED &&\n+\t       ask_yes_no_if_possible(\"Rename from '%s' to '%s' failed. \"\n\nCHECK: Alignment should match open parenthesis\n#163: FILE: compat/precompose_utf8.c:163:\n+\t\t\tstrlcpy(prec_dir->dirent_nfc->d_name, res->d_name,\n+\t\t\t\t\t\t\tprec_dir->dirent_nfc->max_name_len);\n\nCHECK: Alignment should match open parenthesis\n#54: FILE: submodule.c:54:\n+\t\tif (alt_odb->name - alt_odb->base == objects_directory.len &&\n+\t\t\t\t!strncmp(alt_odb->base, objects_directory.buf,\n\nCHECK: Alignment should match open parenthesis\n#213: FILE: submodule.c:213:\n+\t\tadd_pending_object(rev, &list->item->object,\n+\t\t\tsha1_to_hex(list->item->object.sha1));\n\nCHECK: Alignment should match open parenthesis\n#298: FILE: submodule.c:298:\n+\tstrbuf_addf(&sb, \"%s%sSubmodule %s %s..\", line_prefix, meta, path,\n+\t\t\tfind_unique_abbrev(one, DEFAULT_ABBREV));\n\nCHECK: Alignment should match open parenthesis\n#349: FILE: submodule.c:349:\n+\t\t\tdie(\"Could not run 'git rev-list %s --not --remotes -n 1' command\nin submodule %s\",\n+\t\t\t\tsha1_to_hex(sha1), path);\n\nCHECK: Alignment should match open parenthesis\n#865: FILE: submodule.c:865:\n+\tsnprintf(merged_revision, sizeof(merged_revision), \"^%s\",\n+\t\t\tsha1_to_hex(a->object.sha1));\n\nCHECK: Alignment should match open parenthesis\n#124: FILE: notes-merge.c:124:\n+\ttrace_printf(\"\\tdiff_tree_remote(base = %.7s, remote = %.7s)\\n\",\n+\t       sha1_to_hex(base), sha1_to_hex(remote));\n\nCHECK: Alignment should match open parenthesis\n#168: FILE: notes-merge.c:168:\n+\t\ttrace_printf(\"\\t\\tStored remote change for %s: %.7s -> %.7s\\n\",\n+\t\t       sha1_to_hex(mp->obj), sha1_to_hex(mp->base),\n\nCHECK: Alignment should match open parenthesis\n#187: FILE: notes-merge.c:187:\n+\ttrace_printf(\"\\tdiff_tree_local(len = %i, base = %.7s, local = %.7s)\\n\",\n+\t       len, sha1_to_hex(base), sha1_to_hex(local));\n\nCHECK: Alignment should match open parenthesis\n#254: FILE: notes-merge.c:254:\n+\t\ttrace_printf(\"\\t\\tStored local change for %s: %.7s -> %.7s\\n\",\n+\t\t       sha1_to_hex(mp->obj), sha1_to_hex(mp->base),\n\nCHECK: Alignment should match open parenthesis\n#440: FILE: notes-merge.c:440:\n+\t\t\tprintf(\"Using local notes for %s\\n\",\n+\t\t\t\t\t\tsha1_to_hex(p->obj));\n\nCHECK: Alignment should match open parenthesis\n#446: FILE: notes-merge.c:446:\n+\t\t\tprintf(\"Using remote notes for %s\\n\",\n+\t\t\t\t\t\tsha1_to_hex(p->obj));\n\nCHECK: Alignment should match open parenthesis\n#453: FILE: notes-merge.c:453:\n+\t\t\tprintf(\"Concatenating local and remote notes for %s\\n\",\n+\t\t\t\t\t\t\tsha1_to_hex(p->obj));\n\nCHECK: Alignment should match open parenthesis\n#480: FILE: notes-merge.c:480:\n+\t\ttrace_printf(\"\\t\\t%.7s: %.7s -> %.7s/%.7s\\n\",\n+\t\t       sha1_to_hex(p->obj), sha1_to_hex(p->base),\n\nCHECK: Alignment should match open parenthesis\n#574: FILE: notes-merge.c:574:\n+\ttrace_printf(\"notes_merge(o->local_ref = %s, o->remote_ref = %s)\\n\",\n+\t       o->local_ref, o->remote_ref);\n\nCHECK: Alignment should match open parenthesis\n#633: FILE: notes-merge.c:633:\n+\t\t\tprintf(\"One merge base found (%.7s)\\n\",\n+\t\t\t\tsha1_to_hex(base_sha1));\n\nCHECK: Alignment should match open parenthesis\n#680: FILE: notes-merge.c:680:\n+\ttrace_printf(\"notes_merge(): result = %i, result_sha1 = %.7s\\n\",\n+\t       result, sha1_to_hex(result_sha1));\n\nCHECK: Alignment should match open parenthesis\n#706: FILE: notes-merge.c:706:\n+\t\tprintf(\"Committing notes in notes merge worktree at %s\\n\",\n+\t\t\tpath.buf);\n\nCHECK: Alignment should match open parenthesis\n#728: FILE: notes-merge.c:728:\n+\t\t\t\tprintf(\"Skipping non-SHA1 entry '%s%s'\\n\",\n+\t\t\t\t\tpath.buf, e->d_name);\n\nCHECK: Alignment should match open parenthesis\n#743: FILE: notes-merge.c:743:\n+\t\t\tprintf(\"Added resolved note for object %s: %s\\n\",\n+\t\t\t\tsha1_to_hex(obj_sha1), sha1_to_hex(blob_sha1));\n\nCHECK: Alignment should match open parenthesis\n#752: FILE: notes-merge.c:752:\n+\t\tprintf(\"Finalized notes merge commit: %s\\n\",\n+\t\t\tsha1_to_hex(result_sha1));\n\nCHECK: Alignment should match open parenthesis\n#193: FILE: git.c:193:\n+\t\t\tdie_errno(\"While expanding alias '%s': '%s'\",\n+\t\t\t    alias_command, alias_string + 1);\n\nCHECK: Alignment should match open parenthesis\n#205: FILE: git.c:205:\n+\t\tmemmove(new_argv - option_count, new_argv,\n+\t\t\t\tcount * sizeof(char *));\n\nCHECK: Alignment should match open parenthesis\n#835: FILE: dir.c:835:\n+\t\t\tstrcpy(dir->basebuf + stk->baselen,\n+\t\t\t\t\tdir->exclude_per_dir);\n\nCHECK: Alignment should match open parenthesis\n#846: FILE: dir.c:846:\n+\t\t\tadd_excludes_from_file_to_list(dir->basebuf,\n+\t\t\t\t\tdir->basebuf, stk->baselen, el, 1);\n\nCHECK: Alignment should match open parenthesis\n#1283: FILE: dir.c:1283:\n+\t\t\t\texclude_matches_pathspec(path.buf, path.len,\n+\t\t\t\t\tsimplify)))\n\nCHECK: Alignment should match open parenthesis\n#309: FILE: http.c:309:\n+\tcurl_easy_setopt(result, CURLOPT_USERAGENT,\n+\t\tuser_agent ? user_agent : git_user_agent());\n\nCHECK: Alignment should match open parenthesis\n#1068: FILE: http.c:1068:\n+\t\t\t\tfetch_and_setup_pack_index(packs_head, sha1,\n+\t\t\t\t\t\t      base_url);\n\nCHECK: Alignment should match open parenthesis\n#1165: FILE: http.c:1165:\n+\tstrbuf_addf(&buf, \"objects/pack/pack-%s.pack\",\n+\t\tsha1_to_hex(target->sha1));\n\nCHECK: Alignment should match open parenthesis\n#1169: FILE: http.c:1169:\n+\tsnprintf(preq->tmpfile, sizeof(preq->tmpfile), \"%s.temp\",\n+\t\tsha1_pack_name(target->sha1));\n\nCHECK: Alignment should match open parenthesis\n#1182: FILE: http.c:1182:\n+\tcurl_easy_setopt(preq->slot->curl, CURLOPT_HTTPHEADER,\n+\t\tno_pragma_header);\n\nCHECK: Alignment should match open parenthesis\n#1197: FILE: http.c:1197:\n+\t\tcurl_easy_setopt(preq->slot->curl, CURLOPT_HTTPHEADER,\n+\t\t\tpreq->range_header);\n\nCHECK: Alignment should match open parenthesis\n#1331: FILE: http.c:1331:\n+\t\t\t\terror(\"Couldn't truncate temporary file %s: %s\",\n+\t\t\t\t\t  freq->tmpfile, strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#121: FILE: bundle.c:121:\n+\t\tprintf(\"%s %s\\n\", sha1_to_hex(r->list[i].sha1),\n+\t\t\t\tr->list[i].name);\n\nCHECK: Alignment should match open parenthesis\n#176: FILE: bundle.c:176:\n+\t\t\terror(\"%s %s\", sha1_to_hex(refs.objects[i].item->sha1),\n+\t\t\t\trefs.objects[i].name);\n\nCHECK: Alignment should match open parenthesis\n#320: FILE: bundle.c:320:\n+\t\tif (e->item->type == OBJ_TAG &&\n+\t\t\t\t!is_tag_in_date_range(e->item, &revs)) {\n\nCHECK: Alignment should match open parenthesis\n#18: FILE: test-parse-options.c:18:\n+\tprintf(\"Callback: \\\"%s\\\", %d\\n\",\n+\t\t(arg ? arg : \"not set\"), unset);\n\nCHECK: Alignment should match open parenthesis\n#54: FILE: test-parse-options.c:54:\n+\t\tOPT_CALLBACK('L', \"length\", &integer, \"str\",\n+\t\t\t\"get length of <str>\", length_callback),\n\nCHECK: Alignment should match open parenthesis\n#63: FILE: test-parse-options.c:63:\n+\t\tOPT_SET_PTR(0, \"default-string\", &string,\n+\t\t\t\"set string to default\", (unsigned long)\"default\"),\n\nCHECK: Alignment should match open parenthesis\n#68: FILE: test-parse-options.c:68:\n+\t\tOPT_NUMBER_CALLBACK(&integer, \"set integer to NUM\",\n+\t\t\tnumber_callback),\n\nCHECK: Alignment should match open parenthesis\n#101: FILE: date.c:101:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu second ago\", \"%lu seconds ago\", diff), diff);\n\nCHECK: Alignment should match open parenthesis\n#108: FILE: date.c:108:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu minute ago\", \"%lu minutes ago\", diff), diff);\n\nCHECK: Alignment should match open parenthesis\n#115: FILE: date.c:115:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu hour ago\", \"%lu hours ago\", diff), diff);\n\nCHECK: Alignment should match open parenthesis\n#122: FILE: date.c:122:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu day ago\", \"%lu days ago\", diff), diff);\n\nCHECK: Alignment should match open parenthesis\n#128: FILE: date.c:128:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu week ago\", \"%lu weeks ago\", (diff + 3) / 7),\n\nCHECK: Alignment should match open parenthesis\n#135: FILE: date.c:135:\n+\t\tstrbuf_addf(timebuf,\n+\t\t\t Q_(\"%lu month ago\", \"%lu months ago\", (diff + 15) / 30),\n\nCHECK: Alignment should match open parenthesis\n#149: FILE: date.c:149:\n+\t\t\tstrbuf_addf(timebuf,\n+\t\t\t\t Q_(\"%s, %lu month ago\", \"%s, %lu months ago\", months),\n\nCHECK: Alignment should match open parenthesis\n#154: FILE: date.c:154:\n+\t\t\tstrbuf_addf(timebuf,\n+\t\t\t\t Q_(\"%lu year ago\", \"%lu years ago\", years), years);\n\nCHECK: Alignment should match open parenthesis\n#159: FILE: date.c:159:\n+\tstrbuf_addf(timebuf,\n+\t\t Q_(\"%lu year ago\", \"%lu years ago\", (diff + 183) / 365),\n\nCHECK: Alignment should match open parenthesis\n#193: FILE: date.c:193:\n+\t\tstrbuf_addf(&timebuf, \"%04d-%02d-%02d\", tm->tm_year + 1900,\n+\t\t\t\ttm->tm_mon + 1, tm->tm_mday);\n\nCHECK: Alignment should match open parenthesis\n#196: FILE: date.c:196:\n+\t\tstrbuf_addf(&timebuf, \"%04d-%02d-%02d %02d:%02d:%02d %+05d\",\n+\t\t\t\ttm->tm_year + 1900,\n\nCHECK: Alignment should match open parenthesis\n#203: FILE: date.c:203:\n+\t\tstrbuf_addf(&timebuf, \"%.3s, %d %.3s %d %02d:%02d:%02d %+05d\",\n+\t\t\tweekday_names[tm->tm_wday], tm->tm_mday,\n\nCHECK: Alignment should match open parenthesis\n#208: FILE: date.c:208:\n+\t\tstrbuf_addf(&timebuf, \"%.3s %.3s %d %02d:%02d:%02d %d%c%+05d\",\n+\t\t\t\tweekday_names[tm->tm_wday],\n\nCHECK: Alignment should match open parenthesis\n#79: FILE: transport.c:79:\n+\t\t\tif (read_in_full(fd, buffer, 40) != 40 ||\n+\t\t\t\t\tget_sha1_hex(buffer, next->old_sha1)) {\n\nCHECK: Alignment should match open parenthesis\n#158: FILE: transport.c:158:\n+\t\tif (ref->status != REF_STATUS_OK &&\n+\t\t\tref->status != REF_STATUS_UPTODATE)\n\nCHECK: Alignment should match open parenthesis\n#170: FILE: transport.c:170:\n+\t\tif (tmp && flag & REF_ISSYMREF &&\n+\t\t\t!prefixcmp(tmp, \"refs/heads/\"))\n\nCHECK: Alignment should match open parenthesis\n#181: FILE: transport.c:181:\n+\t\t\tinstall_branch_config(BRANCH_CONFIG_VERBOSE,\n+\t\t\t\tlocalname + 11, transport->remote->name,\n\nCHECK: Alignment should match open parenthesis\n#185: FILE: transport.c:185:\n+\t\t\tprintf(\"Would set upstream of '%s' to '%s' of '%s'\\n\",\n+\t\t\t\tlocalname + 11, remotename + 11,\n\nCHECK: Alignment should match open parenthesis\n#298: FILE: transport.c:298:\n+\tif (flags && prefixcmp(name, \"refs/heads/\") &&\n+\t\t\tprefixcmp(name, \"refs/tags/\"))\n\nCHECK: Alignment should match open parenthesis\n#303: FILE: transport.c:303:\n+\tif (safe_create_leading_directories(buf->buf) ||\n+\t\t\t!(f = fopen(buf->buf, \"w\")) ||\n\nCHECK: Alignment should match open parenthesis\n#607: FILE: transport.c:607:\n+\t\t\tupdate_ref(\"update by push\", rs.dst,\n+\t\t\t\t\tref->new_sha1, NULL, 0, 0);\n\nCHECK: Alignment should match open parenthesis\n#649: FILE: transport.c:649:\n+\t\tprint_ref_status('*',\n+\t\t\t(!prefixcmp(ref->name, \"refs/tags/\") ? \"[new tag]\" :\n\nCHECK: Alignment should match open parenthesis\n#684: FILE: transport.c:684:\n+\t\tprint_ref_status('!', \"[rejected]\", ref, NULL,\n+\t\t\t\t\t\t \"remote does not support deleting refs\", porcelain);\n\nCHECK: Alignment should match open parenthesis\n#688: FILE: transport.c:688:\n+\t\tprint_ref_status('=', \"[up to date]\", ref,\n+\t\t\t\t\t\t ref->peer_ref, NULL, porcelain);\n\nCHECK: Alignment should match open parenthesis\n#692: FILE: transport.c:692:\n+\t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n+\t\t\t\t\t\t \"non-fast-forward\", porcelain);\n\nCHECK: Alignment should match open parenthesis\n#696: FILE: transport.c:696:\n+\t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n+\t\t\t\t\t\t \"already exists\", porcelain);\n\nCHECK: Alignment should match open parenthesis\n#700: FILE: transport.c:700:\n+\t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n+\t\t\t\t\t\t \"fetch first\", porcelain);\n\nCHECK: Alignment should match open parenthesis\n#704: FILE: transport.c:704:\n+\t\tprint_ref_status('!', \"[rejected]\", ref, ref->peer_ref,\n+\t\t\t\t\t\t \"needs force\", porcelain);\n\nCHECK: Alignment should match open parenthesis\n#708: FILE: transport.c:708:\n+\t\tprint_ref_status('!', \"[remote rejected]\", ref,\n+\t\t\t\t\t\t ref->deletion ? NULL : ref->peer_ref,\n\nCHECK: Alignment should match open parenthesis\n#713: FILE: transport.c:713:\n+\t\tprint_ref_status('!', \"[remote failure]\", ref,\n+\t\t\t\t\t\t ref->deletion ? NULL : ref->peer_ref,\n\nCHECK: Alignment should match open parenthesis\n#782: FILE: transport.c:782:\n+\t\tif (check_refname_format(remote,\n+\t\t\t\tREFNAME_ALLOW_ONELEVEL|REFNAME_REFSPEC_PATTERN))\n\nCHECK: Alignment should match open parenthesis\n#784: FILE: transport.c:784:\n+\t\t\tdie(\"remote part of refspec is not a valid name in %s\",\n+\t\t\t\theads[i]);\n\nCHECK: Alignment should match open parenthesis\n#1079: FILE: transport.c:1079:\n+\t\tstrbuf_addf( &buf, \"%s %s %s %s\\n\",\n+\t\t\t r->peer_ref->name, sha1_to_hex(r->new_sha1),\n\nCHECK: Alignment should match open parenthesis\n#1140: FILE: transport.c:1140:\n+\t\tset_ref_status_for_push(remote_refs,\n+\t\t\tflags & TRANSPORT_PUSH_MIRROR,\n\nCHECK: Alignment should match open parenthesis\n#1176: FILE: transport.c:1176:\n+\t\t\ttransport_print_push_status(transport->url, remote_refs,\n+\t\t\t\t\tverbose | porcelain, porcelain,\n\nCHECK: Alignment should match open parenthesis\n#724: FILE: builtin/update-index.c:724:\n+\t\tOPT_SET_INT(0, \"add\", &allow_add,\n+\t\t\tN_(\"do not ignore new files\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#726: FILE: builtin/update-index.c:726:\n+\t\tOPT_SET_INT(0, \"replace\", &allow_replace,\n+\t\t\tN_(\"let files replace directories and vice-versa\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#728: FILE: builtin/update-index.c:728:\n+\t\tOPT_SET_INT(0, \"remove\", &allow_remove,\n+\t\t\tN_(\"notice files missing from worktree\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#763: FILE: builtin/update-index.c:763:\n+\t\tOPT_SET_INT(0, \"info-only\", &info_only,\n+\t\t\tN_(\"add to index only; do not add content to object database\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#765: FILE: builtin/update-index.c:765:\n+\t\tOPT_SET_INT(0, \"force-remove\", &force_remove,\n+\t\t\tN_(\"remove named paths even if present in worktree\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#767: FILE: builtin/update-index.c:767:\n+\t\tOPT_SET_INT('z', NULL, &line_termination,\n+\t\t\tN_(\"with --stdin: input lines are terminated by null bytes\"), '\\0'),\n\nCHECK: Alignment should match open parenthesis\n#788: FILE: builtin/update-index.c:788:\n+\t\tOPT_SET_INT(0, \"verbose\", &verbose,\n+\t\t\tN_(\"report actions to standard output\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#794: FILE: builtin/update-index.c:794:\n+\t\tOPT_INTEGER(0, \"index-version\", &preferred_index_format,\n+\t\t\tN_(\"write index in this format\")),\n\nCHECK: Alignment should match open parenthesis\n#719: FILE: builtin/commit.c:719:\n+\t\t\tstatus_printf_ln(s, GIT_COLOR_NORMAL,\n+\t\t\t    whence == FROM_MERGE\n\nCHECK: Alignment should match open parenthesis\n#737: FILE: builtin/commit.c:737:\n+\t\t\tstatus_printf(s, GIT_COLOR_NORMAL,\n+\t\t\t\t_(\"Please enter the commit message for your changes.\"\n\nCHECK: Alignment should match open parenthesis\n#742: FILE: builtin/commit.c:742:\n+\t\t\tstatus_printf(s, GIT_COLOR_NORMAL,\n+\t\t\t\t_(\"Please enter the commit message for your changes.\"\n\nCHECK: Alignment should match open parenthesis\n#749: FILE: builtin/commit.c:749:\n+\t\t\tstatus_printf_ln(s, GIT_COLOR_NORMAL,\n+\t\t\t\t\t\"%s\", only_include_assumed);\n\nCHECK: Alignment should match open parenthesis\n#755: FILE: builtin/commit.c:755:\n+\t\t\tstatus_printf_ln(s, GIT_COLOR_NORMAL,\n+\t\t\t\t_(\"%s\"\n\nCHECK: Alignment should match open parenthesis\n#762: FILE: builtin/commit.c:762:\n+\t\t\tstatus_printf_ln(s, GIT_COLOR_NORMAL,\n+\t\t\t\t_(\"%s\"\n\nCHECK: Alignment should match open parenthesis\n#834: FILE: builtin/commit.c:834:\n+\t\t\tfprintf(stderr,\n+\t\t\t_(\"Please supply the message using either -m or -F option.\\n\"));\n\nCHECK: Alignment should match open parenthesis\n#1288: FILE: builtin/commit.c:1288:\n+\tprintf(\"[%s%s \",\n+\t\t!prefixcmp(head, \"refs/heads/\") ?\n\nCHECK: Alignment should match open parenthesis\n#40: FILE: builtin/symbolic-ref.c:40:\n+\t\tOPT__QUIET(&quiet,\n+\t\t\tN_(\"suppress error message for non-symbolic (detached) refs\")),\n\nCHECK: Alignment should match open parenthesis\n#113: FILE: builtin/mktag.c:113:\n+\tif (!(lb = strstr(tagger_line, \" <\")) || !(rb = strstr(lb+2, \"> \")) ||\n+\t\tstrpbrk(tagger_line, \"<>\\n\") != lb+1 ||\n\nCHECK: Alignment should match open parenthesis\n#492: FILE: builtin/fetch.c:492:\n+\t\t\t\t\tfprintf(stderr, _(\"From %.*s\\n\"),\n+\t\t\t\t\t\t\turl_len, url);\n\nCHECK: Alignment should match open parenthesis\n#677: FILE: builtin/fetch.c:677:\n+\t\tif (ref_map->peer_ref && !strcmp(current_branch->refname,\n+\t\t\t\t\tref_map->peer_ref->name))\n\nCHECK: Alignment should match open parenthesis\n#790: FILE: builtin/fetch.c:790:\n+\t\tdie(_(\"Option \\\"%s\\\" value \\\"%s\\\" is not valid for %s\"),\n+\t\t\tname, value, transport->url);\n\nCHECK: Alignment should match open parenthesis\n#814: FILE: builtin/fetch.c:814:\n+\tif (!prefixcmp(key, \"remotes.\") &&\n+\t\t\t!strcmp(key + 8, g->name)) {\n\nCHECK: Alignment should match open parenthesis\n#92: FILE: builtin/stripspace.c:92:\n+\t\tif (!strcmp(argv[1], \"-s\") ||\n+\t\t\t!strcmp(argv[1], \"--strip-comments\")) {\n\nCHECK: Alignment should match open parenthesis\n#115: FILE: builtin/ls-tree.c:115:\n+\twrite_name_quotedpfx(base + chomp_prefix, baselen - chomp_prefix,\n+\t\t\t  pathname, stdout, line_termination);\n\nCHECK: Alignment should match open parenthesis\n#165: FILE: builtin/mv.c:165:\n+\t\t\t\t\t\tprefix_path(dst, dst_len,\n+\t\t\t\t\t\t\tpath + length + 1);\n\nCHECK: Alignment should match open parenthesis\n#214: FILE: builtin/mv.c:214:\n+\t\tif (!show_only && mode != INDEX &&\n+\t\t\t\trename(src, dst) < 0 && !ignore_errors)\n\nCHECK: Alignment should match open parenthesis\n#57: FILE: builtin/clean.c:57:\n+\tif ((force_flag & REMOVE_DIR_KEEP_NESTED_GIT) &&\n+\t\t\t!resolve_gitlink_ref(path->buf, \"HEAD\", submodule_head)) {\n\nCHECK: Alignment should match open parenthesis\n#61: FILE: builtin/clean.c:61:\n+\t\t\tprintf(dry_run ?  _(msg_would_skip_git_dir) : _(msg_skip_git_dir),\n+\t\t\t\t\tquoted.buf);\n\nCHECK: Alignment should match open parenthesis\n#164: FILE: builtin/clean.c:164:\n+\t\tOPT_BOOLEAN('d', NULL, &remove_directories,\n+\t\t\t\tN_(\"remove whole directories\")),\n\nCHECK: Alignment should match open parenthesis\n#169: FILE: builtin/clean.c:169:\n+\t\tOPT_BOOLEAN('X', NULL, &ignored_only,\n+\t\t\t\tN_(\"remove only ignored files\")),\n\nCHECK: Alignment should match open parenthesis\n#68: FILE: builtin/checkout-index.c:68:\n+\t\tif (checkout_entry(ce, &state,\n+\t\t    to_tempfile ? topath[ce_stage(ce)] : NULL) < 0)\n\nCHECK: Alignment should match open parenthesis\n#112: FILE: builtin/checkout-index.c:112:\n+\t\tif (checkout_entry(ce, &state,\n+\t\t    to_tempfile ? topath[ce_stage(ce)] : NULL) < 0)\n\nCHECK: Alignment should match open parenthesis\n#187: FILE: builtin/checkout-index.c:187:\n+\t\tOPT_BOOLEAN('a', \"all\", &all,\n+\t\t\tN_(\"check out all files in the index\")),\n\nCHECK: Alignment should match open parenthesis\n#190: FILE: builtin/checkout-index.c:190:\n+\t\tOPT__QUIET(&quiet,\n+\t\t\tN_(\"no warning for existing files and files not in index\")),\n\nCHECK: Alignment should match open parenthesis\n#192: FILE: builtin/checkout-index.c:192:\n+\t\tOPT_BOOLEAN('n', \"no-create\", &not_new,\n+\t\t\tN_(\"don't checkout new files\")),\n\nCHECK: Alignment should match open parenthesis\n#200: FILE: builtin/checkout-index.c:200:\n+\t\tOPT_BOOLEAN(0, \"stdin\", &read_from_stdin,\n+\t\t\tN_(\"read list of paths from the standard input\")),\n\nCHECK: Alignment should match open parenthesis\n#202: FILE: builtin/checkout-index.c:202:\n+\t\tOPT_BOOLEAN(0, \"temp\", &to_tempfile,\n+\t\t\tN_(\"write the content to temporary files\")),\n\nCHECK: Alignment should match open parenthesis\n#204: FILE: builtin/checkout-index.c:204:\n+\t\tOPT_CALLBACK(0, \"prefix\", NULL, N_(\"string\"),\n+\t\t\tN_(\"when creating files, prepend <string>\"),\n\nCHECK: Alignment should match open parenthesis\n#207: FILE: builtin/checkout-index.c:207:\n+\t\tOPT_CALLBACK(0, \"stage\", NULL, NULL,\n+\t\t\tN_(\"copy out the files from named stage\"),\n\nCHECK: Alignment should match open parenthesis\n#307: FILE: builtin/index-pack.c:307:\n+\t\tdie(_(\"pack version %\"PRIu32\" unsupported\"),\n+\t\t\tntohl(hdr->hdr_version));\n\nCHECK: Alignment should match open parenthesis\n#1037: FILE: builtin/index-pack.c:1037:\n+\tif (S_ISREG(st.st_mode) &&\n+\t\t\tlseek(input_fd, 0, SEEK_CUR) - input_len != st.st_size)\n\nCHECK: Alignment should match open parenthesis\n#1249: FILE: builtin/index-pack.c:1249:\n+\t\tif (check_sha1_signature(d->base.sha1, base_obj->data,\n+\t\t\t\tbase_obj->size, typename(type)))\n\nCHECK: Alignment should match open parenthesis\n#1296: FILE: builtin/index-pack.c:1296:\n+\t\t\t\tdie_errno(_(\"cannot close written keep file '%s'\"),\n+\t\t\t\t    keep_name);\n\nCHECK: Alignment should match open parenthesis\n#433: FILE: builtin/branch.c:433:\n+\t\t\t\tstrbuf_addf(stat, \"[%s%s%s] \",\n+\t\t\t\t\t\tbranch_get_color(BRANCH_COLOR_UPSTREAM),\n\nCHECK: Alignment should match open parenthesis\n#445: FILE: builtin/branch.c:445:\n+\t\t\tstrbuf_addf(&fancy, \"%s%s%s\",\n+\t\t\t\t\tbranch_get_color(BRANCH_COLOR_UPSTREAM),\n\nCHECK: Alignment should match open parenthesis\n#502: FILE: builtin/branch.c:502:\n+\tstrbuf_addf(out, \" %s %s%s\",\n+\t\tfind_unique_abbrev(item->commit->object.sha1, abbrev),\n\nCHECK: Alignment should match open parenthesis\n#704: FILE: builtin/branch.c:704:\n+\tstrbuf_addf(&logmsg, \"Branch: renamed %s to %s\",\n+\t\t oldref.buf, newref.buf);\n\nCHECK: Alignment should match open parenthesis\n#754: FILE: builtin/branch.c:754:\n+\tstrbuf_commented_addf(&buf,\n+\t\t    \"Please edit the description for the branch\\n\"\n\nCHECK: Alignment should match open parenthesis\n#793: FILE: builtin/branch.c:793:\n+\t\tOPT__VERBOSE(&verbose,\n+\t\t\tN_(\"show hash and subject, give twice for upstream branch\")),\n\nCHECK: Alignment should match open parenthesis\n#796: FILE: builtin/branch.c:796:\n+\t\tOPT_SET_INT('t', \"track\",  &track, N_(\"set up tracking mode (see\ngit-pull(1))\"),\n+\t\t\tBRANCH_TRACK_EXPLICIT),\n\nCHECK: Alignment should match open parenthesis\n#798: FILE: builtin/branch.c:798:\n+\t\tOPT_SET_INT( 0, \"set-upstream\",  &track, N_(\"change upstream info\"),\n+\t\t\tBRANCH_TRACK_OVERRIDE),\n\nCHECK: Alignment should match open parenthesis\n#803: FILE: builtin/branch.c:803:\n+\t\tOPT_SET_INT('r', \"remotes\",     &kinds, N_(\"act on remote-tracking\nbranches\"),\n+\t\t\tREF_REMOTE_BRANCH),\n\nCHECK: Alignment should match open parenthesis\n#820: FILE: builtin/branch.c:820:\n+\t\tOPT_SET_INT('a', \"all\", &kinds, N_(\"list both remote-tracking and\nlocal branches\"),\n+\t\t\tREF_REMOTE_BRANCH | REF_LOCAL_BRANCH),\n\nCHECK: Alignment should match open parenthesis\n#256: FILE: builtin/fast-export.c:256:\n+\t\t\tdie(\"Unexpected comparison status '%c' for %s, %s\",\n+\t\t\t\tq->queue[i]->status,\n\nCHECK: Alignment should match open parenthesis\n#590: FILE: builtin/fast-export.c:590:\n+\t\t\tif (fprintf(f, \":%\"PRIu32\" %s\\n\", mark,\n+\t\t\t\tsha1_to_hex(deco->base->sha1)) < 0) {\n\nCHECK: Alignment should match open parenthesis\n#666: FILE: builtin/fast-export.c:666:\n+\t\tOPT_STRING(0, \"export-marks\", &export_filename, N_(\"file\"),\n+\t\t\t     N_(\"Dump marks to this file\")),\n\nCHECK: Alignment should match open parenthesis\n#668: FILE: builtin/fast-export.c:668:\n+\t\tOPT_STRING(0, \"import-marks\", &import_filename, N_(\"file\"),\n+\t\t\t     N_(\"Import marks from this file\")),\n\nCHECK: Alignment should match open parenthesis\n#670: FILE: builtin/fast-export.c:670:\n+\t\tOPT_BOOLEAN(0, \"fake-missing-tagger\", &fake_missing_tagger,\n+\t\t\t     N_(\"Fake a tagger when tags lack one\")),\n\nCHECK: Alignment should match open parenthesis\n#672: FILE: builtin/fast-export.c:672:\n+\t\tOPT_BOOLEAN(0, \"full-tree\", &full_tree,\n+\t\t\t     N_(\"Output full tree for each commit\")),\n\nCHECK: Alignment should match open parenthesis\n#674: FILE: builtin/fast-export.c:674:\n+\t\tOPT_BOOLEAN(0, \"use-done-feature\", &use_done_feature,\n+\t\t\t     N_(\"Use the done feature to terminate the stream\")),\n\nCHECK: Alignment should match open parenthesis\n#228: FILE: builtin/send-pack.c:228:\n+\tset_ref_status_for_push(remote_refs, args.send_mirror,\n+\t\targs.force_update);\n\nCHECK: Alignment should match open parenthesis\n#90: FILE: builtin/fsck.c:90:\n+\t\tprintf(\"broken link from %7s %s\\n\",\n+\t\t\t   typename(parent->type), sha1_to_hex(parent->sha1));\n\nCHECK: Alignment should match open parenthesis\n#92: FILE: builtin/fsck.c:92:\n+\t\tprintf(\"broken link from %7s %s\\n\",\n+\t\t\t   (type == OBJ_ANY ? \"unknown\" : typename(type)), \"unknown\");\n\nCHECK: Alignment should match open parenthesis\n#107: FILE: builtin/fsck.c:107:\n+\t\t\tprintf(\"broken link from %7s %s\\n\",\n+\t\t\t\t typename(parent->type), sha1_to_hex(parent->sha1));\n\nCHECK: Alignment should match open parenthesis\n#109: FILE: builtin/fsck.c:109:\n+\t\t\tprintf(\"              to %7s %s\\n\",\n+\t\t\t\t typename(obj->type), sha1_to_hex(obj->sha1));\n\nCHECK: Alignment should match open parenthesis\n#623: FILE: builtin/fsck.c:623:\n+\tOPT_BOOLEAN(0, \"lost-found\", &write_lost_and_found,\n+\t\t\t\tN_(\"write dangling objects in .git/lost-found\")),\n\nCHECK: Alignment should match open parenthesis\n#307: FILE: builtin/rev-parse.c:307:\n+\t\tshow_rev(parents_only ? NORMAL : REVERSED,\n+\t\t\t\tparents->item->object.sha1, arg);\n\nCHECK: Alignment should match open parenthesis\n#344: FILE: builtin/rev-parse.c:344:\n+\t\tOPT_BOOLEAN(0, \"keep-dashdash\", &keep_dashdash,\n+\t\t\t\t\tN_(\"keep the `--` passed as an arg\")),\n\nCHECK: Alignment should match open parenthesis\n#346: FILE: builtin/rev-parse.c:346:\n+\t\tOPT_BOOLEAN(0, \"stop-at-non-option\", &stop_at_non_option,\n+\t\t\t\t\tN_(\"stop parsing after the \"\n\nCHECK: Alignment should match open parenthesis\n#621: FILE: builtin/rev-parse.c:621:\n+\t\t\t\tfor_each_glob_ref_in(show_reference, arg + 11,\n+\t\t\t\t\t\"refs/heads/\", NULL);\n\nCHECK: Alignment should match open parenthesis\n#630: FILE: builtin/rev-parse.c:630:\n+\t\t\t\tfor_each_glob_ref_in(show_reference, arg + 7,\n+\t\t\t\t\t\"refs/tags/\", NULL);\n\nCHECK: Alignment should match open parenthesis\n#643: FILE: builtin/rev-parse.c:643:\n+\t\t\t\tfor_each_glob_ref_in(show_reference, arg + 10,\n+\t\t\t\t\t\"refs/remotes/\", NULL);\n\nCHECK: Alignment should match open parenthesis\n#56: FILE: builtin/rerere.c:56:\n+\t\tOPT_SET_INT(0, \"rerere-autoupdate\", &autoupdate,\n+\t\t\tN_(\"register clean resolutions in index\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#415: FILE: builtin/describe.c:415:\n+\t\tOPT_BOOLEAN(0, \"always\",     &always,\n+\t\t\t   N_(\"show abbreviated commit object as fallback\")),\n\nCHECK: Alignment should match open parenthesis\n#294: FILE: builtin/shortlog.c:294:\n+\t\tqsort(log->list.items, log->list.nr, sizeof(struct string_list_item),\n+\t\t\tcompare_by_number);\n\nCHECK: Alignment should match open parenthesis\n#432: FILE: builtin/fmt-merge-msg.c:432:\n+\t\t\tprint_joined(\"branch \", \"branches \", &src_data->branch,\n+\t\t\t\t\tout);\n\nCHECK: Alignment should match open parenthesis\n#438: FILE: builtin/fmt-merge-msg.c:438:\n+\t\t\tprint_joined(\"remote-tracking branch \", \"remote-tracking branches \",\n+\t\t\t\t\t&src_data->r_branch, out);\n\nCHECK: Alignment should match open parenthesis\n#448: FILE: builtin/fmt-merge-msg.c:448:\n+\t\t\tprint_joined(\"commit \", \"commits \", &src_data->generic,\n+\t\t\t\t\tout);\n\nCHECK: Alignment should match open parenthesis\n#508: FILE: builtin/fmt-merge-msg.c:508:\n+\t\t\t\tstrbuf_add_commented_lines(&tagline,\n+\t\t\t\t\t\torigins.items[first_tag].string,\n\nCHECK: Alignment should match open parenthesis\n#516: FILE: builtin/fmt-merge-msg.c:516:\n+\t\t\tstrbuf_add_commented_lines(&tagbuf,\n+\t\t\t\t\torigins.items[i].string,\n\nCHECK: Alignment should match open parenthesis\n#671: FILE: builtin/fmt-merge-msg.c:671:\n+\t\tOPT_STRING('m', \"message\", &message, N_(\"text\"),\n+\t\t\tN_(\"use <text> as start of message\")),\n\nCHECK: Alignment should match open parenthesis\n#1006: FILE: builtin/for-each-ref.c:1006:\n+\t\tOPT_CALLBACK(0 , \"sort\", sort_tail, N_(\"key\"),\n+\t\t\t    N_(\"field name to sort on\"), &opt_parse_sort),\n\nCHECK: Alignment should match open parenthesis\n#75: FILE: builtin/clone.c:75:\n+\tOPT_BOOL('l', \"local\", &option_local,\n+\t\tN_(\"to clone from a local repository\")),\n\nCHECK: Alignment should match open parenthesis\n#95: FILE: builtin/clone.c:95:\n+\tOPT_STRING(0, \"depth\", &option_depth, N_(\"depth\"),\n+\t\t    N_(\"create a shallow clone of that depth\")),\n\nCHECK: Alignment should match open parenthesis\n#97: FILE: builtin/clone.c:97:\n+\tOPT_BOOL(0, \"single-branch\", &option_single_branch,\n+\t\t    N_(\"clone only one branch, HEAD or --branch\")),\n\nCHECK: Alignment should match open parenthesis\n#684: FILE: builtin/clone.c:684:\n+\t\t\t\t\tstrbuf_addf(&value, \"+%s:%s\", our_head_points_at->name,\n+\t\t\t\t\t\tour_head_points_at->name);\n\nCHECK: Alignment should match open parenthesis\n#687: FILE: builtin/clone.c:687:\n+\t\t\t\t\tstrbuf_addf(&value, \"+%s:%s%s\", our_head_points_at->name,\n+\t\t\t\t\t\tbranch_top->buf, option_branch);\n\nCHECK: Alignment should match open parenthesis\n#690: FILE: builtin/clone.c:690:\n+\t\t\t\tstrbuf_addf(&value, \"+%s:%s%s\", remote_head_points_at->name,\n+\t\t\t\t\t\tbranch_top->buf,\n\nCHECK: Alignment should match open parenthesis\n#750: FILE: builtin/clone.c:750:\n+\t\tusage_msg_opt(_(\"Too many arguments.\"),\n+\t\t\tbuiltin_clone_usage, builtin_clone_options);\n\nCHECK: Alignment should match open parenthesis\n#754: FILE: builtin/clone.c:754:\n+\t\tusage_msg_opt(_(\"You must specify a repository to clone.\"),\n+\t\t\tbuiltin_clone_usage, builtin_clone_options);\n\nCHECK: Alignment should match open parenthesis\n#945: FILE: builtin/clone.c:945:\n+\twrite_refspec_config(src_ref_prefix, our_head_points_at,\n+\t\t\tremote_head_points_at, &branch_top);\n\nCHECK: Alignment should match open parenthesis\n#479: FILE: builtin/unpack-objects.c:479:\n+\t\tdie(\"unknown pack file version %\"PRIu32,\n+\t\t\tntohl(hdr->hdr_version));\n\nCHECK: Alignment should match open parenthesis\n#250: FILE: builtin/reset.c:250:\n+\t\tOPT_SET_INT(0, \"mixed\", &reset_type,\n+\t\t\t\t\t\tN_(\"reset HEAD and index\"), MIXED),\n\nCHECK: Alignment should match open parenthesis\n#253: FILE: builtin/reset.c:253:\n+\t\tOPT_SET_INT(0, \"hard\", &reset_type,\n+\t\t\t\tN_(\"reset HEAD, index and working tree\"), HARD),\n\nCHECK: Alignment should match open parenthesis\n#255: FILE: builtin/reset.c:255:\n+\t\tOPT_SET_INT(0, \"merge\", &reset_type,\n+\t\t\t\tN_(\"reset HEAD, index and working tree\"), MERGE),\n\nCHECK: Alignment should match open parenthesis\n#257: FILE: builtin/reset.c:257:\n+\t\tOPT_SET_INT(0, \"keep\", &reset_type,\n+\t\t\t\tN_(\"reset HEAD but keep local changes\"), KEEP),\n\nCHECK: Alignment should match open parenthesis\n#304: FILE: builtin/reset.c:304:\n+\t\t\tdie(_(\"Cannot do %s reset with paths.\"),\n+\t\t\t\t\t_(reset_type_names[reset_type]));\n\nCHECK: Alignment should match open parenthesis\n#122: FILE: builtin/remote.c:122:\n+\t\tstrbuf_addf(tmp, \"refs/%s:refs/%s\",\n+\t\t\t\tbranchname, branchname);\n\nCHECK: Alignment should match open parenthesis\n#125: FILE: builtin/remote.c:125:\n+\t\tstrbuf_addf(tmp, \"refs/heads/%s:refs/remotes/%s/%s\",\n+\t\t\t\tbranchname, remotename, branchname);\n\nCHECK: Alignment should match open parenthesis\n#194: FILE: builtin/remote.c:194:\n+\tif (remote && (remote->url_nr > 1 || strcmp(name, remote->url[0]) ||\n+\t\t\tremote->fetch_refspec_nr))\n\nCHECK: Alignment should match open parenthesis\n#228: FILE: builtin/remote.c:228:\n+\t\tif (git_config_set(buf.buf,\n+\t\t\tfetch_tags == TAGS_SET ? \"--tags\" : \"--no-tags\"))\n\nCHECK: Alignment should match open parenthesis\n#340: FILE: builtin/remote.c:340:\n+\t\t\tdie(_(\"Could not get fetch map for refspec %s\"),\n+\t\t\t\tstates->remote->fetch_refspec[i]);\n\nCHECK: Alignment should match open parenthesis\n#726: FILE: builtin/remote.c:726:\n+\t\tstrbuf_splice(&buf, strlen(\"refs/remotes/\"), strlen(rename.old),\n+\t\t\t\trename.new, strlen(rename.new));\n\nCHECK: Alignment should match open parenthesis\n#729: FILE: builtin/remote.c:729:\n+\t\tstrbuf_addf(&buf2, \"remote: renamed %s to %s\",\n+\t\t\t\titem->string, buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#741: FILE: builtin/remote.c:741:\n+\t\tstrbuf_splice(&buf, strlen(\"refs/remotes/\"), strlen(rename.old),\n+\t\t\t\trename.new, strlen(rename.new));\n\nCHECK: Alignment should match open parenthesis\n#745: FILE: builtin/remote.c:745:\n+\t\tstrbuf_splice(&buf2, strlen(\"refs/remotes/\"), strlen(rename.old),\n+\t\t\t\trename.new, strlen(rename.new));\n\nCHECK: Alignment should match open parenthesis\n#748: FILE: builtin/remote.c:748:\n+\t\tstrbuf_addf(&buf3, \"remote: renamed %s to %s\",\n+\t\t\t\titem->string, buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#810: FILE: builtin/remote.c:810:\n+\t\t\t\tstrbuf_addf(&buf, \"branch.%s.%s\",\n+\t\t\t\t\t\titem->string, *k);\n\nCHECK: Alignment should match open parenthesis\n#992: FILE: builtin/remote.c:992:\n+\t\terror(_(\"invalid branch.%s.merge; cannot rebase onto > 1 branch\"),\n+\t\t\titem->string);\n\nCHECK: Alignment should match open parenthesis\n#1072: FILE: builtin/remote.c:1072:\n+\t\t\tprintf_ln(_(\"    %-*s forces to %-*s (%s)\"), show_info->width, src,\n+\t\t\t       show_info->width2, push_info->dest, status);\n\nCHECK: Alignment should match open parenthesis\n#1075: FILE: builtin/remote.c:1075:\n+\t\t\tprintf_ln(_(\"    %-*s pushes to %-*s (%s)\"), show_info->width, src,\n+\t\t\t       show_info->width2, push_info->dest, status);\n\nCHECK: Alignment should match open parenthesis\n#1079: FILE: builtin/remote.c:1079:\n+\t\t\tprintf_ln(_(\"    %-*s forces to %s\"), show_info->width, src,\n+\t\t\t       push_info->dest);\n\nCHECK: Alignment should match open parenthesis\n#1082: FILE: builtin/remote.c:1082:\n+\t\t\tprintf_ln(_(\"    %-*s pushes to %s\"), show_info->width, src,\n+\t\t\t       push_info->dest);\n\nCHECK: Alignment should match open parenthesis\n#1176: FILE: builtin/remote.c:1176:\n+\t\tqsort(info.list->items, info.list->nr,\n+\t\t\tsizeof(*info.list->items), cmp_string_with_push);\n\nCHECK: Alignment should match open parenthesis\n#1283: FILE: builtin/remote.c:1283:\n+\t\tprintf_ln(_(\"URL: %s\"),\n+\t\t       states.remote->url_nr\n\nCHECK: Alignment should match open parenthesis\n#1296: FILE: builtin/remote.c:1296:\n+\t\t\tprintf_ln(_(\" * [would prune] %s\"),\n+\t\t\t       abbrev_ref(refname, \"refs/remotes/\"));\n\nCHECK: Alignment should match open parenthesis\n#1299: FILE: builtin/remote.c:1299:\n+\t\t\tprintf_ln(_(\" * [pruned] %s\"),\n+\t\t\t       abbrev_ref(refname, \"refs/remotes/\"));\n\nCHECK: Alignment should match open parenthesis\n#1478: FILE: builtin/remote.c:1478:\n+\t\t\tgit_config_set_multivar(name_buf.buf, newurl,\n+\t\t\t\t\"^$\", 0);\n\nCHECK: Alignment should match open parenthesis\n#1554: FILE: builtin/remote.c:1554:\n+\t\t\t\tprintf(\"%s\\t%s\\n\", item->string,\n+\t\t\t\t\titem->util ? (const char *)item->util : \"\");\n\nCHECK: Alignment should match open parenthesis\n#2068: FILE: builtin/apply.c:2068:\n+\tprintf(\"%5d %.*s%.*s\\n\", patch->lines_added + patch->lines_deleted,\n+\t\tadd, pluses, del, minuses);\n\nCHECK: Alignment should match open parenthesis\n#2318: FILE: builtin/apply.c:2318:\n+\t\tupdate_pre_post_images(preimage, postimage,\n+\t\t\t\tfixed_buf, fixed_len, postlen);\n\nCHECK: Alignment should match open parenthesis\n#4366: FILE: builtin/apply.c:4366:\n+\t\tOPT_BOOLEAN(0, \"no-add\", &no_add,\n+\t\t\tN_(\"ignore additions made by the patch\")),\n\nCHECK: Alignment should match open parenthesis\n#4368: FILE: builtin/apply.c:4368:\n+\t\tOPT_BOOLEAN(0, \"stat\", &diffstat,\n+\t\t\tN_(\"instead of applying the patch, output diffstat for the input\")),\n\nCHECK: Alignment should match open parenthesis\n#4372: FILE: builtin/apply.c:4372:\n+\t\tOPT_BOOLEAN(0, \"numstat\", &numstat,\n+\t\t\tN_(\"show number of added and deleted lines in decimal notation\")),\n\nCHECK: Alignment should match open parenthesis\n#4374: FILE: builtin/apply.c:4374:\n+\t\tOPT_BOOLEAN(0, \"summary\", &summary,\n+\t\t\tN_(\"instead of applying the patch, output a summary for the input\")),\n\nCHECK: Alignment should match open parenthesis\n#4376: FILE: builtin/apply.c:4376:\n+\t\tOPT_BOOLEAN(0, \"check\", &check,\n+\t\t\tN_(\"instead of applying the patch, see if the patch is applicable\")),\n\nCHECK: Alignment should match open parenthesis\n#4378: FILE: builtin/apply.c:4378:\n+\t\tOPT_BOOLEAN(0, \"index\", &check_index,\n+\t\t\tN_(\"make sure the patch is applicable to the current index\")),\n\nCHECK: Alignment should match open parenthesis\n#4380: FILE: builtin/apply.c:4380:\n+\t\tOPT_BOOLEAN(0, \"cached\", &cached,\n+\t\t\tN_(\"apply a patch without touching the working tree\")),\n\nCHECK: Alignment should match open parenthesis\n#4382: FILE: builtin/apply.c:4382:\n+\t\tOPT_BOOLEAN(0, \"apply\", &force_apply,\n+\t\t\tN_(\"also apply the patch (use with --stat/--summary/--check)\")),\n\nCHECK: Alignment should match open parenthesis\n#4386: FILE: builtin/apply.c:4386:\n+\t\tOPT_FILENAME(0, \"build-fake-ancestor\", &fake_ancestor,\n+\t\t\tN_(\"build a temporary index based on embedded index information\")),\n\nCHECK: Alignment should match open parenthesis\n#4391: FILE: builtin/apply.c:4391:\n+\t\tOPT_INTEGER('C', NULL, &p_context,\n+\t\t\t\tN_(\"ensure at least <n> lines of context match\")),\n\nCHECK: Alignment should match open parenthesis\n#4402: FILE: builtin/apply.c:4402:\n+\t\tOPT_BOOLEAN('R', \"reverse\", &apply_in_reverse,\n+\t\t\tN_(\"apply the patch in reverse\")),\n\nCHECK: Alignment should match open parenthesis\n#4404: FILE: builtin/apply.c:4404:\n+\t\tOPT_BOOLEAN(0, \"unidiff-zero\", &unidiff_zero,\n+\t\t\tN_(\"don't expect at least one line of context\")),\n\nCHECK: Alignment should match open parenthesis\n#4406: FILE: builtin/apply.c:4406:\n+\t\tOPT_BOOLEAN(0, \"reject\", &apply_with_reject,\n+\t\t\tN_(\"leave the rejected hunks in corresponding *.rej files\")),\n\nCHECK: Alignment should match open parenthesis\n#4408: FILE: builtin/apply.c:4408:\n+\t\tOPT_BOOLEAN(0, \"allow-overlap\", &allow_overlap,\n+\t\t\tN_(\"allow overlapping hunks\")),\n\nCHECK: Alignment should match open parenthesis\n#406: FILE: builtin/push.c:406:\n+\t\tdie(\"option %s needs an argument (check|on-demand)\",\n+\t\t\t\topt->long_name);\n\nCHECK: Alignment should match open parenthesis\n#422: FILE: builtin/push.c:422:\n+\t\tOPT_BIT( 0 , \"mirror\", &flags, N_(\"mirror all refs\"),\n+\t\t\t    (TRANSPORT_PUSH_MIRROR|TRANSPORT_PUSH_FORCE)),\n\nCHECK: Alignment should match open parenthesis\n#44: FILE: builtin/remote-fd.c:44:\n+\t\t\tif (bidirectional_transfer_loop(input_fd,\n+\t\t\t\toutput_fd))\n\nCHECK: Alignment should match open parenthesis\n#52: FILE: builtin/ls-files.c:52:\n+\twrite_name_quoted_relative(name, len, prefix, prefix_len, stdout,\n+\t\t\tline_terminator);\n\nCHECK: Alignment should match open parenthesis\n#459: FILE: builtin/ls-files.c:459:\n+\t\tOPT_BOOLEAN('t', NULL, &show_tag,\n+\t\t\tN_(\"identify the file status with tags\")),\n\nCHECK: Alignment should match open parenthesis\n#461: FILE: builtin/ls-files.c:461:\n+\t\tOPT_BOOLEAN('v', NULL, &show_valid_bit,\n+\t\t\tN_(\"use lowercase letters for 'assume unchanged' files\")),\n\nCHECK: Alignment should match open parenthesis\n#463: FILE: builtin/ls-files.c:463:\n+\t\tOPT_BOOLEAN('c', \"cached\", &show_cached,\n+\t\t\tN_(\"show cached files in the output (default)\")),\n\nCHECK: Alignment should match open parenthesis\n#465: FILE: builtin/ls-files.c:465:\n+\t\tOPT_BOOLEAN('d', \"deleted\", &show_deleted,\n+\t\t\tN_(\"show deleted files in the output\")),\n\nCHECK: Alignment should match open parenthesis\n#467: FILE: builtin/ls-files.c:467:\n+\t\tOPT_BOOLEAN('m', \"modified\", &show_modified,\n+\t\t\tN_(\"show modified files in the output\")),\n\nCHECK: Alignment should match open parenthesis\n#469: FILE: builtin/ls-files.c:469:\n+\t\tOPT_BOOLEAN('o', \"others\", &show_others,\n+\t\t\tN_(\"show other files in the output\")),\n\nCHECK: Alignment should match open parenthesis\n#474: FILE: builtin/ls-files.c:474:\n+\t\tOPT_BOOLEAN('s', \"stage\", &show_stage,\n+\t\t\tN_(\"show staged contents' object name in the output\")),\n\nCHECK: Alignment should match open parenthesis\n#476: FILE: builtin/ls-files.c:476:\n+\t\tOPT_BOOLEAN('k', \"killed\", &show_killed,\n+\t\t\tN_(\"show files on the filesystem that need to be removed\")),\n\nCHECK: Alignment should match open parenthesis\n#481: FILE: builtin/ls-files.c:481:\n+\t\tOPT_NEGBIT(0, \"empty-directory\", &dir.flags,\n+\t\t\tN_(\"don't show empty directories\"),\n\nCHECK: Alignment should match open parenthesis\n#484: FILE: builtin/ls-files.c:484:\n+\t\tOPT_BOOLEAN('u', \"unmerged\", &show_unmerged,\n+\t\t\tN_(\"show unmerged files in the output\")),\n\nCHECK: Alignment should match open parenthesis\n#494: FILE: builtin/ls-files.c:494:\n+\t\tOPT_STRING(0, \"exclude-per-directory\", &dir.exclude_per_dir, N_(\"file\"),\n+\t\t\tN_(\"read additional per-directory exclude patterns in <file>\")),\n\nCHECK: Alignment should match open parenthesis\n#502: FILE: builtin/ls-files.c:502:\n+\t\tOPT_BOOLEAN(0, \"error-unmatch\", &error_unmatch,\n+\t\t\tN_(\"if any <file> is not in the index, treat this as an error\")),\n\nCHECK: Alignment should match open parenthesis\n#504: FILE: builtin/ls-files.c:504:\n+\t\tOPT_STRING(0, \"with-tree\", &with_tree, N_(\"tree-ish\"),\n+\t\t\tN_(\"pretend that paths removed since <tree-ish> are still present\")),\n\nCHECK: Alignment should match open parenthesis\n#505: FILE: builtin/log.c:505:\n+\t\t\tprintf(\"%stag %s%s\\n\",\n+\t\t\t\t\tdiff_get_color_opt(&rev.diffopt, DIFF_COMMIT),\n\nCHECK: Alignment should match open parenthesis\n#524: FILE: builtin/log.c:524:\n+\t\t\tprintf(\"%stree %s%s\\n\\n\",\n+\t\t\t\t\tdiff_get_color_opt(&rev.diffopt, DIFF_COMMIT),\n\nCHECK: Alignment should match open parenthesis\n#528: FILE: builtin/log.c:528:\n+\t\t\tread_tree_recursive((struct tree *)o, \"\", 0, 0, &match_all,\n+\t\t\t\t\tshow_tree_object, NULL);\n\nCHECK: Alignment should match open parenthesis\n#778: FILE: builtin/log.c:778:\n+\tclear_commit_marks((struct commit *)o1,\n+\t\t\tSEEN | UNINTERESTING | SHOWN | ADDED);\n\nCHECK: Alignment should match open parenthesis\n#780: FILE: builtin/log.c:780:\n+\tclear_commit_marks((struct commit *)o2,\n+\t\t\tSEEN | UNINTERESTING | SHOWN | ADDED);\n\nCHECK: Alignment should match open parenthesis\n#1115: FILE: builtin/log.c:1115:\n+\t\tOPT_BOOL(0, \"stdout\", &use_stdout,\n+\t\t\t    N_(\"print patches to standard out\")),\n\nCHECK: Alignment should match open parenthesis\n#1117: FILE: builtin/log.c:1117:\n+\t\tOPT_BOOL(0, \"cover-letter\", &cover_letter,\n+\t\t\t    N_(\"generate a cover letter\")),\n\nCHECK: Alignment should match open parenthesis\n#1119: FILE: builtin/log.c:1119:\n+\t\tOPT_BOOL(0, \"numbered-files\", &just_numbers,\n+\t\t\t    N_(\"use simple number sequence for output file names\")),\n\nCHECK: Alignment should match open parenthesis\n#1121: FILE: builtin/log.c:1121:\n+\t\tOPT_STRING(0, \"suffix\", &fmt_patch_suffix, N_(\"sfx\"),\n+\t\t\t    N_(\"use <sfx> instead of '.patch'\")),\n\nCHECK: Alignment should match open parenthesis\n#1150: FILE: builtin/log.c:1150:\n+\t\tOPT_STRING(0, \"in-reply-to\", &in_reply_to, N_(\"message-id\"),\n+\t\t\t    N_(\"make first mail a reply to <message-id>\")),\n\nCHECK: Alignment should match open parenthesis\n#1162: FILE: builtin/log.c:1162:\n+\t\tOPT_STRING(0, \"signature\", &signature, N_(\"signature\"),\n+\t\t\t    N_(\"add a signature\")),\n\nCHECK: Alignment should match open parenthesis\n#1265: FILE: builtin/log.c:1265:\n+\tif (!use_patch_format &&\n+\t\t(!rev.diffopt.output_format ||\n\nCHECK: Alignment should match open parenthesis\n#1354: FILE: builtin/log.c:1354:\n+\t\tif (ignore_if_in_upstream &&\n+\t\t\t\thas_commit_patch_id(commit, &ids))\n\nCHECK: Alignment should match open parenthesis\n#222: FILE: builtin/gc.c:222:\n+\t\t\tfprintf(stderr,\n+\t\t\t\t\t_(\"Auto packing the repository for optimum performance. You may also\\n\"\n\nCHECK: Alignment should match open parenthesis\n#556: FILE: builtin/checkout.c:556:\n+\t\t\tmerge_trees(&o, new->commit->tree, work,\n+\t\t\t\told->commit->tree, &result);\n\nCHECK: Alignment should match open parenthesis\n#602: FILE: builtin/checkout.c:602:\n+\t\t\t\t\tfprintf(stderr, _(\"Can not do reflog for '%s'\\n\"),\n+\t\t\t\t\t    opts->new_orphan_branch);\n\nCHECK: Alignment should match open parenthesis\n#685: FILE: builtin/checkout.c:685:\n+\tstrbuf_addstr(sb,\n+\t\tfind_unique_abbrev(commit->object.sha1, DEFAULT_ABBREV));\n\nCHECK: Alignment should match open parenthesis\n#1052: FILE: builtin/checkout.c:1052:\n+\t\tOPT_SET_INT('t', \"track\",  &opts.track, N_(\"set upstream info for\nnew branch\"),\n+\t\t\tBRANCH_TRACK_EXPLICIT),\n\nCHECK: Alignment should match open parenthesis\n#89: FILE: builtin/archive.c:89:\n+\t\tOPT_STRING('o', \"output\", &output, N_(\"file\"),\n+\t\t\tN_(\"write the archive to this file\")),\n\nCHECK: Alignment should match open parenthesis\n#91: FILE: builtin/archive.c:91:\n+\t\tOPT_STRING(0, \"remote\", &remote, N_(\"repo\"),\n+\t\t\tN_(\"retrieve the archive from remote repository <repo>\")),\n\nCHECK: Alignment should match open parenthesis\n#93: FILE: builtin/archive.c:93:\n+\t\tOPT_STRING(0, \"exec\", &exec, N_(\"command\"),\n+\t\t\tN_(\"path to the remote git-upload-archive command\")),\n\nCHECK: Alignment should match open parenthesis\n#186: FILE: builtin/merge.c:186:\n+\tOPT_BOOLEAN(0, \"stat\", &show_diffstat,\n+\t\tN_(\"show a diffstat at the end of the merge\")),\n\nCHECK: Alignment should match open parenthesis\n#192: FILE: builtin/merge.c:192:\n+\tOPT_BOOLEAN(0, \"squash\", &squash,\n+\t\tN_(\"create a single commit instead of doing a merge\")),\n\nCHECK: Alignment should match open parenthesis\n#194: FILE: builtin/merge.c:194:\n+\tOPT_BOOLEAN(0, \"commit\", &option_commit,\n+\t\tN_(\"perform a commit if the merge succeeds (default)\")),\n\nCHECK: Alignment should match open parenthesis\n#196: FILE: builtin/merge.c:196:\n+\tOPT_BOOL('e', \"edit\", &option_edit,\n+\t\tN_(\"edit message before committing\")),\n\nCHECK: Alignment should match open parenthesis\n#198: FILE: builtin/merge.c:198:\n+\tOPT_BOOLEAN(0, \"ff\", &allow_fast_forward,\n+\t\tN_(\"allow fast-forward (default)\")),\n\nCHECK: Alignment should match open parenthesis\n#200: FILE: builtin/merge.c:200:\n+\tOPT_BOOLEAN(0, \"ff-only\", &fast_forward_only,\n+\t\tN_(\"abort if fast-forward is not possible\")),\n\nCHECK: Alignment should match open parenthesis\n#203: FILE: builtin/merge.c:203:\n+\tOPT_BOOL(0, \"verify-signatures\", &verify_signatures,\n+\t\tN_(\"Verify that the named commit has a valid GPG signature\")),\n\nCHECK: Alignment should match open parenthesis\n#205: FILE: builtin/merge.c:205:\n+\tOPT_CALLBACK('s', \"strategy\", &use_strategies, N_(\"strategy\"),\n+\t\tN_(\"merge strategy to use\"), option_parse_strategy),\n\nCHECK: Alignment should match open parenthesis\n#207: FILE: builtin/merge.c:207:\n+\tOPT_CALLBACK('X', \"strategy-option\", &xopts, N_(\"option=value\"),\n+\t\tN_(\"option for selected merge strategy\"), option_parse_x),\n\nCHECK: Alignment should match open parenthesis\n#209: FILE: builtin/merge.c:209:\n+\tOPT_CALLBACK('m', \"message\", &merge_msg, N_(\"message\"),\n+\t\tN_(\"merge commit message (for a non-fast-forward merge)\"),\n\nCHECK: Alignment should match open parenthesis\n#213: FILE: builtin/merge.c:213:\n+\tOPT_BOOLEAN(0, \"abort\", &abort_current_merge,\n+\t\tN_(\"abort the current in-progress merge\")),\n\nCHECK: Alignment should match open parenthesis\n#359: FILE: builtin/merge.c:359:\n+\t\tstrbuf_addf(&out, \"commit %s\\n\",\n+\t\t\tsha1_to_hex(commit->object.sha1));\n\nCHECK: Alignment should match open parenthesis\n#382: FILE: builtin/merge.c:382:\n+\t\tstrbuf_addf(&reflog_message, \"%s: %s\",\n+\t\t\tgetenv(\"GIT_REFLOG_ACTION\"), msg);\n\nCHECK: Alignment should match open parenthesis\n#392: FILE: builtin/merge.c:392:\n+\t\t\tupdate_ref(reflog_message.buf, \"HEAD\",\n+\t\t\t\tnew_head, head, 0,\n\nCHECK: Alignment should match open parenthesis\n#501: FILE: builtin/merge.c:501:\n+\tif (!strcmp(remote, \"FETCH_HEAD\") &&\n+\t\t\t!access(git_path(\"FETCH_HEAD\"), R_OK)) {\n\nCHECK: Alignment should match open parenthesis\n#535: FILE: builtin/merge.c:535:\n+\tstrbuf_addf(msg, \"%s\\t\\tcommit '%s'\\n\",\n+\t\tsha1_to_hex(remote_head->object.sha1), remote);\n\nCHECK: Alignment should match open parenthesis\n#566: FILE: builtin/merge.c:566:\n+\tif (branch && !prefixcmp(k, \"branch.\") &&\n+\t\t!prefixcmp(k + 7, branch) &&\n\nCHECK: Alignment should match open parenthesis\n#656: FILE: builtin/merge.c:656:\n+\tif (active_cache_changed &&\n+\t\t\t(write_cache(index_fd, active_cache, active_nr) ||\n\nCHECK: Alignment should match open parenthesis\n#697: FILE: builtin/merge.c:697:\n+\t\tif (active_cache_changed &&\n+\t\t\t\t(write_cache(index_fd, active_cache, active_nr) ||\n\nCHECK: Alignment should match open parenthesis\n#1178: FILE: builtin/merge.c:1178:\n+\t\tusage_with_options(builtin_merge_usage,\n+\t\t\tbuiltin_merge_options);\n\nCHECK: Alignment should match open parenthesis\n#1249: FILE: builtin/merge.c:1249:\n+\t\tusage_with_options(builtin_merge_usage,\n+\t\t\tbuiltin_merge_options);\n\nCHECK: Alignment should match open parenthesis\n#1359: FILE: builtin/merge.c:1359:\n+\t\t\tprintf(_(\"Updating %s..%s\\n\"),\n+\t\t\t\thex,\n\nCHECK: Alignment should match open parenthesis\n#1361: FILE: builtin/merge.c:1361:\n+\t\t\t\tfind_unique_abbrev(remoteheads->item->object.sha1,\n+\t\t\t\tDEFAULT_ABBREV));\n\nCHECK: Alignment should match open parenthesis\n#1365: FILE: builtin/merge.c:1365:\n+\t\t\tstrbuf_addstr(&msg,\n+\t\t\t\t\" (no commit created; -m option ignored)\");\n\nCHECK: Alignment should match open parenthesis\n#1424: FILE: builtin/merge.c:1424:\n+\t\t\tif (hashcmp(common_one->item->object.sha1,\n+\t\t\t\tj->item->object.sha1)) {\n\nCHECK: Alignment should match open parenthesis\n#1464: FILE: builtin/merge.c:1464:\n+\t\t\tprintf(_(\"Trying merge strategy %s...\\n\"),\n+\t\t\t\tuse_strategies[i]->name);\n\nCHECK: Alignment should match open parenthesis\n#1541: FILE: builtin/merge.c:1541:\n+\t\tprintf(_(\"Using the %s to prepare resolving by hand.\\n\"),\n+\t\t\tbest_strategy);\n\nCHECK: Alignment should match open parenthesis\n#47: FILE: builtin/help.c:47:\n+\tOPT_SET_INT('w', \"web\", &help_format, N_(\"show manual in web browser\"),\n+\t\t\tHELP_FORMAT_WEB),\n\nCHECK: Alignment should match open parenthesis\n#49: FILE: builtin/help.c:49:\n+\tOPT_SET_INT('i', \"info\", &help_format, N_(\"show info page\"),\n+\t\t\tHELP_FORMAT_INFO),\n\nCHECK: Alignment should match open parenthesis\n#53: FILE: builtin/hash-object.c:53:\n+\t\thash_object(buf.buf, type, write_objects,\n+\t\t    no_filters ? NULL : buf.buf);\n\nCHECK: Alignment should match open parenthesis\n#835: FILE: builtin/receive-pack.c:835:\n+\tsnprintf(hdr_arg, sizeof(hdr_arg),\n+\t\t\t\"--pack_header=%\"PRIu32\",%\"PRIu32,\n\nCHECK: Alignment should match open parenthesis\n#114: FILE: builtin/revert.c:114:\n+\t\tOPT_CALLBACK('X', \"strategy-option\", &opts, N_(\"option\"),\n+\t\t\tN_(\"option for merge strategy\"), option_parse_x),\n\nCHECK: Alignment should match open parenthesis\n#142: FILE: builtin/revert.c:142:\n+\tverify_opt_mutually_compatible(me,\n+\t\t\t\t\"--quit\", remove_state,\n\nCHECK: Alignment should match open parenthesis\n#174: FILE: builtin/revert.c:174:\n+\t\tverify_opt_compatible(me, this_operation,\n+\t\t\t\t\"--no-commit\", opts->no_commit,\n\nCHECK: Alignment should match open parenthesis\n#186: FILE: builtin/revert.c:186:\n+\t\tverify_opt_compatible(me, \"--ff\",\n+\t\t\t\t\"--signoff\", opts->signoff,\n\nCHECK: Alignment should match open parenthesis\n#57: FILE: builtin/diff-tree.c:57:\n+\tprintf(\"%s %s\\n\", sha1_to_hex(tree1->object.sha1),\n+\t\t\t  sha1_to_hex(tree2->object.sha1));\n\nCHECK: Alignment should match open parenthesis\n#486: FILE: builtin/init-db.c:486:\n+\t\tOPT_STRING(0, \"template\", &template_dir, N_(\"template-directory\"),\n+\t\t\t\tN_(\"directory from which templates will be used\")),\n\nCHECK: Alignment should match open parenthesis\n#488: FILE: builtin/init-db.c:488:\n+\t\tOPT_SET_INT(0, \"bare\", &is_bare_repository_cfg,\n+\t\t\t\tN_(\"create a bare repository\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#542: FILE: builtin/init-db.c:542:\n+\t\tsetenv(GIT_DIR_ENVIRONMENT,\n+\t\t\tgetcwd(git_dir, sizeof(git_dir)), argc > 0);\n\nCHECK: Alignment should match open parenthesis\n#71: FILE: builtin/name-rev.c:71:\n+\t\t\t\tsprintf(new_name, \"%.*s~%d^%d\", len, tip_name,\n+\t\t\t\t\t\tgeneration, parent_number);\n\nCHECK: Alignment should match open parenthesis\n#74: FILE: builtin/name-rev.c:74:\n+\t\t\t\tsprintf(new_name, \"%.*s^%d\", len, tip_name,\n+\t\t\t\t\t\tparent_number);\n\nCHECK: Alignment should match open parenthesis\n#77: FILE: builtin/name-rev.c:77:\n+\t\t\tname_rev(parents->item, new_name, 0,\n+\t\t\t\tdistance + MERGE_TRAVERSAL_WEIGHT, 0);\n\nCHECK: Alignment should match open parenthesis\n#80: FILE: builtin/name-rev.c:80:\n+\t\t\tname_rev(parents->item, tip_name, generation + 1,\n+\t\t\t\tdistance + 1, 0);\n\nCHECK: Alignment should match open parenthesis\n#148: FILE: builtin/name-rev.c:148:\n+\t\tsnprintf(buffer, sizeof(buffer), \"%.*s~%d\", len, n->tip_name,\n+\t\t\t\tn->generation);\n\nCHECK: Alignment should match open parenthesis\n#232: FILE: builtin/name-rev.c:232:\n+\t\tOPT_STRING(0, \"refs\", &data.ref_filter, N_(\"pattern\"),\n+\t\t\t\t   N_(\"only use refs matching <pattern>\")),\n\nCHECK: Alignment should match open parenthesis\n#238: FILE: builtin/name-rev.c:238:\n+\t\tOPT_BOOLEAN(0, \"always\",     &always,\n+\t\t\t   N_(\"show abbreviated commit object as fallback\")),\n\nCHECK: Alignment should match open parenthesis\n#258: FILE: builtin/name-rev.c:258:\n+\t\t\tfprintf(stderr, \"Could not get sha1 for %s. Skipping.\\n\",\n+\t\t\t\t\t*argv);\n\nCHECK: Alignment should match open parenthesis\n#265: FILE: builtin/name-rev.c:265:\n+\t\t\tfprintf(stderr, \"Could not get commit for %s. Skipping.\\n\",\n+\t\t\t\t\t*argv);\n\nCHECK: Alignment should match open parenthesis\n#39: FILE: builtin/mailinfo.c:39:\n+\tif (name->len < 3 || 60 < name->len || strchr(name->buf, '@') ||\n+\t\tstrchr(name->buf, '<') || strchr(name->buf, '>'))\n\nCHECK: Alignment should match open parenthesis\n#643: FILE: builtin/grep.c:643:\n+\t\tOPT_BOOLEAN(0, \"cached\", &cached,\n+\t\t\tN_(\"search in index instead of in the work tree\")),\n\nCHECK: Alignment should match open parenthesis\n#645: FILE: builtin/grep.c:645:\n+\t\tOPT_NEGBIT(0, \"no-index\", &use_index,\n+\t\t\t N_(\"find in contents not managed by git\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#647: FILE: builtin/grep.c:647:\n+\t\tOPT_BOOLEAN(0, \"untracked\", &untracked,\n+\t\t\tN_(\"search in both tracked and untracked files\")),\n\nCHECK: Alignment should match open parenthesis\n#652: FILE: builtin/grep.c:652:\n+\t\tOPT_BOOLEAN('v', \"invert-match\", &opt.invert,\n+\t\t\tN_(\"show non-matching lines\")),\n\nCHECK: Alignment should match open parenthesis\n#654: FILE: builtin/grep.c:654:\n+\t\tOPT_BOOLEAN('i', \"ignore-case\", &opt.ignore_case,\n+\t\t\tN_(\"case insensitive matching\")),\n\nCHECK: Alignment should match open parenthesis\n#656: FILE: builtin/grep.c:656:\n+\t\tOPT_BOOLEAN('w', \"word-regexp\", &opt.word_regexp,\n+\t\t\tN_(\"match patterns only at word boundaries\")),\n\nCHECK: Alignment should match open parenthesis\n#658: FILE: builtin/grep.c:658:\n+\t\tOPT_SET_INT('a', \"text\", &opt.binary,\n+\t\t\tN_(\"process binary files as text\"), GREP_BINARY_TEXT),\n\nCHECK: Alignment should match open parenthesis\n#660: FILE: builtin/grep.c:660:\n+\t\tOPT_SET_INT('I', NULL, &opt.binary,\n+\t\t\tN_(\"don't match patterns in binary files\"),\n\nCHECK: Alignment should match open parenthesis\n#683: FILE: builtin/grep.c:683:\n+\t\tOPT_NEGBIT(0, \"full-name\", &opt.relative,\n+\t\t\tN_(\"show filenames relative to top directory\"), 1),\n\nCHECK: Alignment should match open parenthesis\n#685: FILE: builtin/grep.c:685:\n+\t\tOPT_BOOLEAN('l', \"files-with-matches\", &opt.name_only,\n+\t\t\tN_(\"show only filenames instead of matching lines\")),\n\nCHECK: Alignment should match open parenthesis\n#687: FILE: builtin/grep.c:687:\n+\t\tOPT_BOOLEAN(0, \"name-only\", &opt.name_only,\n+\t\t\tN_(\"synonym for --files-with-matches\")),\n\nCHECK: Alignment should match open parenthesis\n#689: FILE: builtin/grep.c:689:\n+\t\tOPT_BOOLEAN('L', \"files-without-match\",\n+\t\t\t&opt.unmatch_name_only,\n\nCHECK: Alignment should match open parenthesis\n#692: FILE: builtin/grep.c:692:\n+\t\tOPT_BOOLEAN('z', \"null\", &opt.null_following_name,\n+\t\t\tN_(\"print NUL after filenames\")),\n\nCHECK: Alignment should match open parenthesis\n#694: FILE: builtin/grep.c:694:\n+\t\tOPT_BOOLEAN('c', \"count\", &opt.count,\n+\t\t\tN_(\"show the number of matches instead of matching lines\")),\n\nCHECK: Alignment should match open parenthesis\n#697: FILE: builtin/grep.c:697:\n+\t\tOPT_BOOLEAN(0, \"break\", &opt.file_break,\n+\t\t\tN_(\"print empty line between matches from different files\")),\n\nCHECK: Alignment should match open parenthesis\n#699: FILE: builtin/grep.c:699:\n+\t\tOPT_BOOLEAN(0, \"heading\", &opt.heading,\n+\t\t\tN_(\"show filename only once above matches from same file\")),\n\nCHECK: Alignment should match open parenthesis\n#702: FILE: builtin/grep.c:702:\n+\t\tOPT_CALLBACK('C', \"context\", &opt, N_(\"n\"),\n+\t\t\tN_(\"show <n> context lines before and after matches\"),\n\nCHECK: Alignment should match open parenthesis\n#705: FILE: builtin/grep.c:705:\n+\t\tOPT_INTEGER('B', \"before-context\", &opt.pre_context,\n+\t\t\tN_(\"show <n> context lines before matches\")),\n\nCHECK: Alignment should match open parenthesis\n#707: FILE: builtin/grep.c:707:\n+\t\tOPT_INTEGER('A', \"after-context\", &opt.post_context,\n+\t\t\tN_(\"show <n> context lines after matches\")),\n\nCHECK: Alignment should match open parenthesis\n#709: FILE: builtin/grep.c:709:\n+\t\tOPT_NUMBER_CALLBACK(&opt, N_(\"shortcut for -C NUM\"),\n+\t\t\tcontext_callback),\n\nCHECK: Alignment should match open parenthesis\n#711: FILE: builtin/grep.c:711:\n+\t\tOPT_BOOLEAN('p', \"show-function\", &opt.funcname,\n+\t\t\tN_(\"show a line with the function name before matches\")),\n\nCHECK: Alignment should match open parenthesis\n#713: FILE: builtin/grep.c:713:\n+\t\tOPT_BOOLEAN('W', \"function-context\", &opt.funcbody,\n+\t\t\tN_(\"show the surrounding function\")),\n\nCHECK: Alignment should match open parenthesis\n#716: FILE: builtin/grep.c:716:\n+\t\tOPT_CALLBACK('f', NULL, &opt, N_(\"file\"),\n+\t\t\tN_(\"read patterns from file\"), file_callback),\n\nCHECK: Alignment should match open parenthesis\n#734: FILE: builtin/grep.c:734:\n+\t\tOPT_BOOLEAN(0, \"all-match\", &opt.all_match,\n+\t\t\tN_(\"show only matches from files that match all patterns\")),\n\nCHECK: Alignment should match open parenthesis\n#878: FILE: builtin/grep.c:878:\n+\t\t\tstrbuf_addf(&buf, \"+/%s%s\",\n+\t\t\t\t\tstrcmp(\"less\", pager) ? \"\" : \"*\",\n\nCHECK: Alignment should match open parenthesis\n#74: FILE: builtin/mailsplit.c:74:\n+\t\tif (!keep_cr && buf.len > 1 && buf.buf[buf.len-1] == '\\n' &&\n+\t\t\tbuf.buf[buf.len-2] == '\\r') {\n\nCHECK: Alignment should match open parenthesis\n#359: FILE: builtin/tag.c:359:\n+\t\t\tfprintf(stderr,\n+\t\t\t_(\"Please supply the message using either -m or -F option.\\n\"));\n\nCHECK: Alignment should match open parenthesis\n#454: FILE: builtin/tag.c:454:\n+\t\tOPT_BOOLEAN('a', \"annotate\", &annotate,\n+\t\t\t\t\tN_(\"annotated tag, needs a message\")),\n\nCHECK: Alignment should match open parenthesis\n#460: FILE: builtin/tag.c:460:\n+\t\tOPT_STRING(0, \"cleanup\", &cleanup_arg, N_(\"mode\"),\n+\t\t\tN_(\"how to strip spaces and #comments from message\")),\n\nCHECK: Alignment should match open parenthesis\n#462: FILE: builtin/tag.c:462:\n+\t\tOPT_STRING('u', \"local-user\", &keyid, N_(\"key id\"),\n+\t\t\t\t\tN_(\"use another key to sign the tag\")),\n\nCHECK: Alignment should match open parenthesis\n#544: FILE: builtin/tag.c:544:\n+\t\t\t\t\tdie_errno(_(\"could not open or read '%s'\"),\n+\t\t\t\t\t\tmsgfile);\n\nCHECK: Alignment should match open parenthesis\n#21: FILE: builtin/update-ref.c:21:\n+\t\tOPT_BOOLEAN( 0 , \"no-deref\", &no_deref,\n+\t\t\t\t\tN_(\"update <refname> not the one it points to\")),\n\nCHECK: Alignment should match open parenthesis\n#812: FILE: builtin/pack-objects.c:812:\n+\t\tdie(\"wrote %\"PRIu32\" objects while expecting %\"PRIu32,\n+\t\t\twritten, nr_result);\n\nCHECK: Alignment should match open parenthesis\n#1912: FILE: builtin/pack-objects.c:1912:\n+\t\tfprintf(stderr, \"Delta compression using up to %d threads.\\n\",\n+\t\t\t\tdelta_search_threads);\n\nCHECK: Alignment should match open parenthesis\n#2311: FILE: builtin/pack-objects.c:2311:\n+\t\tif ((!p->pack_local || p->pack_keep) &&\n+\t\t\tfind_pack_entry_one(sha1, p)) {\n\nCHECK: Alignment should match open parenthesis\n#2345: FILE: builtin/pack-objects.c:2345:\n+\t\t\tif (!locate_object_entry(sha1) &&\n+\t\t\t\t!has_sha1_pack_kept_or_nonlocal(sha1))\n\nCHECK: Alignment should match open parenthesis\n#23: FILE: builtin/check-attr.c:23:\n+\tOPT_BOOLEAN('z', NULL, &null_term_line,\n+\t\tN_(\"input paths are terminated by a null character\")),\n\nCHECK: Alignment should match open parenthesis\n#61: FILE: builtin/remote-ext.c:61:\n+\t\t\t\tdie(\"Bad remote-ext placeholder '%%%c'.\",\n+\t\t\t\t\tstr[rpos]);\n\nCHECK: Alignment should match open parenthesis\n#117: FILE: builtin/mktree.c:117:\n+\t\tdie(\"entry '%s' object type (%s) doesn't match mode type (%s)\",\n+\t\t\tpath, ptr, typename(mode_type));\n\nCHECK: Alignment should match open parenthesis\n#136: FILE: builtin/mktree.c:136:\n+\t\t\tdie(\"entry '%s' object %s is a %s but specified type was (%s)\",\n+\t\t\t\tpath, sha1_to_hex(sha1), typename(obj_type), typename(mode_type));\n\nCHECK: Alignment should match open parenthesis\n#212: FILE: builtin/rm.c:212:\n+\tOPT_BOOLEAN( 0 , \"ignore-unmatch\", &ignore_unmatch,\n+\t\t\t\tN_(\"exit with a zero status even if nothing matched\")),\n\nCHECK: Alignment should match open parenthesis\n#377: FILE: builtin/notes.c:377:\n+\t\t\terror(_(\"Bad %s value: '%s'\"), GIT_NOTES_REWRITE_MODE_ENVIRONMENT,\n+\t\t\t\t\trewrite_mode_env);\n\nCHECK: Alignment should match open parenthesis\n#654: FILE: builtin/show-branch.c:654:\n+\t\tOPT__COLOR(&showbranch_use_color,\n+\t\t\t    N_(\"color '*!+-' corresponding to the branch\")),\n\nCHECK: Alignment should match open parenthesis\n#293: FILE: archive-zip.c:293:\n+\tcopy_le16(dirent.creator_version,\n+\t\tS_ISLNK(mode) || (S_ISREG(mode) && (mode & 0111)) ? 0x0317 : 0);\n\nCHECK: Alignment should match open parenthesis\n#345: FILE: mailmap.c:345:\n+\t\tdebug_mm(\"map_user:  to '%.*s' <.*%s>\\n\", *namelen, *name,\n+\t\t\t\t *emaillen, *email);\n\nCHECK: Alignment should match open parenthesis\n#438: FILE: log-tree.c:438:\n+\t\tstrbuf_addf(&verify_message, \"tag %s names a non-parent %s\\n\",\n+\t\t\t\t    tag->tag, tag->tagged->sha1);\n\nCHECK: Alignment should match open parenthesis\n#57: FILE: alloc.c:57:\n+\tfprintf(stderr, \"%10s: %8u (%\"PRIuMAX\" kB)\\n\",\n+\t\t\tname, count, (uintmax_t) size);\n\nCHECK: Alignment should match open parenthesis\n#327: FILE: run-command.c:327:\n+\t\t\terror(\"cannot create %s pipe for %s: %s\",\n+\t\t\t\tstr, cmd->argv[0], strerror(failed_errno));\n\nCHECK: Alignment should match open parenthesis\n#397: FILE: run-command.c:397:\n+\t\t\tdie_errno(\"exec '%s': cd to '%s' failed\", cmd->argv[0],\n+\t\t\t    cmd->dir);\n\nCHECK: Alignment should match open parenthesis\n#416: FILE: run-command.c:416:\n+\t\t\t\terror(\"cannot run %s: %s\", cmd->argv[0],\n+\t\t\t\t\tstrerror(ENOENT));\n\nCHECK: Alignment should match open parenthesis\n#424: FILE: run-command.c:424:\n+\t\terror(\"cannot fork() for %s: %s\", cmd->argv[0],\n+\t\t\tstrerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#45: FILE: string-list.c:45:\n+\t\tmemmove(list->items + index + 1, list->items + index,\n+\t\t\t\t(list->nr - index)\n\nCHECK: Alignment should match open parenthesis\n#116: FILE: usage.c:116:\n+\t\tfputs(\"fatal: recursion detected in die_errno handler\\n\",\n+\t\t\tstderr);\n\nCHECK: Alignment should match open parenthesis\n#425: FILE: notes.c:425:\n+\t\tdie(\"Could not read %s for notes-index\",\n+\t\t     sha1_to_hex(subtree->val_sha1));\n\nCHECK: Alignment should match open parenthesis\n#1218: FILE: notes.c:1218:\n+\tif (!(msg = read_sha1_file(sha1, &type, &msglen)) || !msglen ||\n+\t\t\ttype != OBJ_BLOB) {\n\nCHECK: Alignment should match open parenthesis\n#347: FILE: connect.c:347:\n+\t\t\tstrbuf_addf(&error_message, \"%s[%d: %s]: errno=%s\\n\",\n+\t\t\t\thost,\n\nCHECK: Alignment should match open parenthesis\n#80: FILE: refs.c:80:\n+\t\t\tif ((flags & REFNAME_REFSPEC_PATTERN) &&\n+\t\t\t\t\trefname[0] == '*' &&\n\nCHECK: Alignment should match open parenthesis\n#902: FILE: refs.c:902:\n+\tadd_ref(get_packed_refs(get_ref_cache(NULL)),\n+\t\t\tcreate_ref_entry(refname, sha1, REF_ISPACKED, 1));\n\nCHECK: Alignment should match open parenthesis\n#1138: FILE: refs.c:1138:\n+\t\t\tif (!prefixcmp(buffer, \"refs/\") &&\n+\t\t\t\t\t!check_refname_format(buffer, 0)) {\n\nCHECK: Alignment should match open parenthesis\n#1546: FILE: refs.c:1546:\n+\t\terror(\"Ref %s is at %s but expected %s\", lock->ref_name,\n+\t\t\tsha1_to_hex(lock->old_sha1), sha1_to_hex(old_sha1));\n\nCHECK: Alignment should match open parenthesis\n#1695: FILE: refs.c:1695:\n+\t\terror(\"unable to resolve reference %s: %s\",\n+\t\t\torig_refname, strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#1705: FILE: refs.c:1705:\n+\tif (missing &&\n+\t     !is_refname_available(refname, NULL,\nget_packed_refs(get_ref_cache(NULL)))) {\n\nCHECK: Alignment should match open parenthesis\n#1908: FILE: refs.c:1908:\n+\t\t\terror(\"unable to move logfile \"TMP_RENAMED_LOG\" to logs/%s: %s\",\n+\t\t\t\tnewrefname, strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#1945: FILE: refs.c:1945:\n+\t\terror(\"unable to restore logfile %s from %s: %s\",\n+\t\t\toldrefname, newrefname, strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#1949: FILE: refs.c:1949:\n+\t\terror(\"unable to restore logfile %s from \"TMP_RENAMED_LOG\": %s\",\n+\t\t\toldrefname, strerror(errno));\n\nCHECK: Alignment should match open parenthesis\n#2102: FILE: refs.c:2102:\n+\t\terror(\"Trying to write ref %s with nonexistent object %s\",\n+\t\t\tlock->ref_name, sha1_to_hex(sha1));\n\nCHECK: Alignment should match open parenthesis\n#2108: FILE: refs.c:2108:\n+\t\terror(\"Trying to write non-commit object %s to branch %s\",\n+\t\t\tsha1_to_hex(sha1), lock->ref_name);\n---\n\nThere are 535 instances where parenthesis are not aligned. Do you\nstill think aligning open parenthesis is the \"style of the project\"?\n\nPlease. You are rejecting this patch on a whim, and nothing more.\n\nDo you know what I do in the projects I maintain? I tell people to run\ncheckpatch, if checkpatch doesn't have any problems with the style,\nneither do I. To do anything else is a waste of style for me, and the\ncontributors, because going back and forth into arguing if the style\nis violated or not, and discussing every two patches what should be\nthe ideal guideline, and if the patch is meting the guideline or not\nis *NOT PRODUCTIVE*. I do not tell them to follow the style of the\nsurrounding code, because the surrounding code might have the wrong\nstyle, and they are perpetuating it.\n\nBut do what you will. I don't care. This patch fixed the code to\nfollow Documentation/CodingGuidelines to the letter, and it even\npasses Linux's checkpatch script. Any sensible maintainer would have\napplied it without reservations. Period.\n\n-- \nFelipe Contreras\n"},{"id":"217666","messageId":"CAMP44s1bvCYw+srX_geY-4YeY7pBVVE=fC5O+BScZfQDhC27UQ@mail.gmail.com","threadId":"33756","inReplyTo":"CALkWK0=LuycRpuwBMjfiSk9qy2Y6VZsmmhNLpKdovKX4rZ=hTA@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T23:58:32Z","receivedAt":"2013-05-16T23:58:32Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 1:34 PM, Ramkumar Ramachandra\n<artagnon@gmail.com> wrote:\n> Junio C Hamano wrote:\n>>     (defun linux-style ()\n>>       \"C mode with adjusted defaults for use with the Linux kernel.\"\n>>       (interactive)\n>>       (c-set-style \"K&R\")\n>\n> There's actually a \"linux\" style you can inherit from.\n>\n> May I suggest looking at Documentation/CodingStyle in linux.git for this:\n\nSurprise surprise, we have a discussion about what should be the\ncoding style. Didn't Junio say it was already \"the style of the\nproject\"? Moreover, I don't speak emacs, so it's all gibberish to me.\n\n-- \nFelipe Contreras\n"},{"id":"217670","messageId":"CAMP44s0PYreS7g8FfUW-3+9xY56-yiusKRcmUKHnZc4kDwXF-A@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s2QcFStPtE8cSbH7jWvUOUVRSgGNYNVoDN6exRdW6xNYA@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-17T00:24:55Z","receivedAt":"2013-05-17T00:24:55Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 6:54 PM, Felipe Contreras\n<felipe.contreras@gmail.com> wrote:\n\n> There are 535 instances where parenthesis are not aligned. Do you\n> still think aligning open parenthesis is the \"style of the project\"?\n>\n> Please. You are rejecting this patch on a whim, and nothing more.\n\nAnd FTR. I shouldn't need to waste my time demonstrating this obvious\nfact to you, you should already know that the project doesn't follow\nthis style, and if you don't know for sure, you shouldn't act as if\nyou do. And even if you were correct, and this was the style of the\nproject, you should not reject patches based on unstated guidelines.\n\nWould you think it's fair for you to go to jail for breaking a \"law\"\nthat is not written anywhere?\n\n-- \nFelipe Contreras\n"},{"id":"217676","messageId":"CALkWK0k6D+hOw2iQ=tJ+_bkKOA-7dKkEsG9XBSBqkg9hDm-bGA@mail.gmail.com","threadId":"33756","inReplyTo":"CAMP44s2QcFStPtE8cSbH7jWvUOUVRSgGNYNVoDN6exRdW6xNYA@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2013-05-17T06:04:55Z","receivedAt":"2013-05-17T06:04:55Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Felipe Contreras wrote:\n> This is what Linux's checkpatch throws:\n\nPerhaps it's time to include checkpatch.pl in our tree and update our\nCodingGuidelines?\n"},{"id":"217677","messageId":"CAMP44s2yOGtwz6Ez6aHSQKQi+gZWf1dvU__sf9ssJf52g3MLFw@mail.gmail.com","threadId":"33756","inReplyTo":"CALkWK0k6D+hOw2iQ=tJ+_bkKOA-7dKkEsG9XBSBqkg9hDm-bGA@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-17T06:09:57Z","receivedAt":"2013-05-17T06:09:57Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Fri, May 17, 2013 at 1:04 AM, Ramkumar Ramachandra\n<artagnon@gmail.com> wrote:\n> Felipe Contreras wrote:\n>> This is what Linux's checkpatch throws:\n>\n> Perhaps it's time to include checkpatch.pl in our tree and update our\n> CodingGuidelines?\n\ncheckpatch only works with a Linux tree, and it doesn't treat this\nalignment as an error, not even a warning, because Linux's style\ndoesn't have it specified either. In Linux, maintainers couldn't\npossibly reject a patch because of this.\n\n-- \nFelipe Contreras\n"},{"id":"217721","messageId":"7vobc91squ.fsf@alter.siamese.dyndns.org","threadId":"33756","inReplyTo":"CAMP44s2QcFStPtE8cSbH7jWvUOUVRSgGNYNVoDN6exRdW6xNYA@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-17T16:22:17Z","receivedAt":"2013-05-17T16:22:17Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> *You* are telling my that; it's *your* opinion and nothing else. It's\n\nI saw a review comment that points out that the continuation lines\ndo not align, and you refused to say \"ah, thanks for spotting\" and\nreroll [*1*], so even I do not want to do so in general, I had to\nplay the role of the arbiter.\n\nMy take on these style issues is this:\n\n * People made mistakes in the past while doing real work.  Big\n   news: contributors and reviewers are not perfect.\n\n * They survived to this day because we do not do tree-wide \"style\n   fixes\" for the sake of style fix, in order to avoid clashing with\n   real work in flight.\n\n * Existing mistakes are not an excuse for adding new mistakes of\n   the same kind, especially when they are pointed out during the\n   review (this is not limited to \"style issues\").\n\nI do not think I would reject a patch with minor style bugs like\nunaligned continuation lines, if it were a patch that does real\nwork.\n\nBut a \"style cleanups\" patch that introduces new instances of style\nbreakage is a different matter.  It is clear that the original\n(picked randomly):\n\n        die (\"Encountered signed tag %s; use \",\n             \"--signed-tags=<mode> to handle it.\",\n             sha1_to_hex(tag->object.sha1));\n\nwanted the opening double-quotes of two lines and the \"sha1\" at the\nbeginning of the third line to align.  I see that is the local style\na \"style cleanup\" change should follow.\n\nA patch that cleans up styles in preparation for a real work (like\nthis one) is a rare and precious occasion for us to really clean up\naccumulated wart.  I do not want to see existing mistakes from other\nunrelated parts of the codebase that have not been cleaned up as an\nexcuse to waste that rare occasion to do a good job of cleaning up.\n\nSo that is the arbiter's decision.  Call it *my* opinion or whatever\nyou like; it does not change anything.\n\n\n[Footnote]\n\n*1* That would have ended this thread without wasting everybody's\ntime.\n"},{"id":"217722","messageId":"CAMP44s0o7tgUrz4xQh3H62+=625ppAOMFskOL70Nrx-O5uwaYw@mail.gmail.com","threadId":"33756","inReplyTo":"7vobc91squ.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-17T16:35:48Z","receivedAt":"2013-05-17T16:35:48Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Fri, May 17, 2013 at 11:22 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> *You* are telling my that; it's *your* opinion and nothing else. It's\n>\n> I saw a review comment that points out that the continuation lines\n> do not align, and you refused to say \"ah, thanks for spotting\" and\n> reroll [*1*], so even I do not want to do so in general, I had to\n> play the role of the arbiter.\n\nSpotting what? There no style issues in my patch.\n\n> My take on these style issues is this:\n>\n>  * People made mistakes in the past while doing real work.  Big\n>    news: contributors and reviewers are not perfect.\n\nThat's exactly why \"follow the surrounding code\" is not a good guideline.\n\n>  * They survived to this day because we do not do tree-wide \"style\n>    fixes\" for the sake of style fix, in order to avoid clashing with\n>    real work in flight.\n>\n>  * Existing mistakes are not an excuse for adding new mistakes of\n>    the same kind, especially when they are pointed out during the\n>    review (this is not limited to \"style issues\").\n\nFortunately nobody is adding new mistakes in this patch.\n\n> I do not think I would reject a patch with minor style bugs like\n> unaligned continuation lines, if it were a patch that does real\n> work.\n>\n> But a \"style cleanups\" patch that introduces new instances of style\n> breakage is a different matter.\n\nTHERE IS NO STYLE BREAKAGE.\n\n> It is clear that the original\n> (picked randomly):\n>\n>         die (\"Encountered signed tag %s; use \",\n>              \"--signed-tags=<mode> to handle it.\",\n>              sha1_to_hex(tag->object.sha1));\n>\n> wanted the opening double-quotes of two lines and the \"sha1\" at the\n> beginning of the third line to align.  I see that is the local style\n> a \"style cleanup\" change should follow.\n\nUnless the original had a mistaken style, or there was no guideline at\nall, which is the case.\n\n> A patch that cleans up styles in preparation for a real work (like\n> this one) is a rare and precious occasion for us to really clean up\n> accumulated wart.  I do not want to see existing mistakes from other\n> unrelated parts of the codebase that have not been cleaned up as an\n> excuse to waste that rare occasion to do a good job of cleaning up.\n\nThis is a good job of cleaning up. It follows the coding style PERFECTLY.\n\n> So that is the arbiter's decision.  Call it *my* opinion or whatever\n> you like; it does not change anything.\n\nYou are rejecting a patch for not following your *ARBITRARY* coding\nstyle, not the project's.\n\nAll you are doing is using rhetoric to say that this patch has \"broken\nstyle\" doesn't follow the \"project's style\", and whatever, and you\nconveniently ignore the fact that 535 instances of this style, and the\nfact that it's not mentioned at all in Documentation/CodingStyle\nproves that it does indeed follow the project's style.\n\nKeep using rhetoric all you want, it doesn't change the facts. You are\nrejecting the patch because of your own personal subjective reasons,\nand not because of anything related to the project's style.\n\n> [Footnote]\n>\n> *1* That would have ended this thread without wasting everybody's\n> time.\n\nIt would have ended if you applied the patch that follows the coding\nstyle to the letter.\n\n-- \nFelipe Contreras\n"},{"id":"217726","messageId":"vpqvc6hczq2.fsf@grenoble-inp.fr","threadId":"33756","inReplyTo":"CAMP44s0o7tgUrz4xQh3H62+=625ppAOMFskOL70Nrx-O5uwaYw@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2013-05-17T16:56:05Z","receivedAt":"2013-05-17T16:56:05Z","isPatch":true,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> THERE IS NO STYLE BREAKAGE.\n\nRepeating something, and even making it all caps does not make it true.\n\nYou are wasting your time and everybody else's in this thread and many\nothers. You seem to reject the very concept of code review. The Git\ncommunity likes very much code review. I do not forsee a change in the\ntaste for code review and iterations to make the code as clean as it can\nbe in the Git community. And I'd hate to see it change because some\ncontributors are too stubborn to send a reroll when someone else points\nout something in the patch.\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"},{"id":"217730","messageId":"CAMP44s2AGWk76Nwk+va-6SBZqj46AHf6d_tm-AeN8jWKR-mEvw@mail.gmail.com","threadId":"33756","inReplyTo":"vpqvc6hczq2.fsf@grenoble-inp.fr","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-17T17:02:58Z","receivedAt":"2013-05-17T17:02:58Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Fri, May 17, 2013 at 11:56 AM, Matthieu Moy\n<Matthieu.Moy@grenoble-inp.fr> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> THERE IS NO STYLE BREAKAGE.\n>\n> Repeating something, and even making it all caps does not make it true.\n\nTell that to Junio. He is also repeating the same.\n\nI actually provided *evidence*. Did you miss it?\n\n> You are wasting your time and everybody else's in this thread and many\n> others. You seem to reject the very concept of code review. The Git\n> community likes very much code review. I do not forsee a change in the\n> taste for code review and iterations to make the code as clean as it can\n> be in the Git community. And I'd hate to see it change because some\n> contributors are too stubborn to send a reroll when someone else points\n> out something in the patch.\n\nSorry, did I miss somebody mentioning a problem in this patch? All\nI've seen is people throwing opinions, nothing related to this\nproject's coding style.\n\nIf you see this patch doesn't follow the project's style, please point\nwhere, and where it's described in Documentation/CodingStyle.\n\n-- \nFelipe Contreras\n"},{"id":"217734","messageId":"vpqwqqxbka7.fsf@grenoble-inp.fr","threadId":"33756","inReplyTo":"CAMP44s2AGWk76Nwk+va-6SBZqj46AHf6d_tm-AeN8jWKR-mEvw@mail.gmail.com","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Matthieu Moy","fromEmail":"matthieu.moy@grenoble-inp.fr","sentAt":"2013-05-17T17:14:56Z","receivedAt":"2013-05-17T17:14:56Z","isPatch":true,"sender":{"key":"matthieu.moy@grenoble-inp.fr","avatar":"https://gravatar.com/avatar/72c8a2705971a25dfaff23cece15130d405685845d911aedd5667ace277f3fc5?d=mp&s=160"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> On Fri, May 17, 2013 at 11:56 AM, Matthieu Moy\n> <Matthieu.Moy@grenoble-inp.fr> wrote:\n>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>\n>>> THERE IS NO STYLE BREAKAGE.\n>>\n>> Repeating something, and even making it all caps does not make it true.\n>\n> Tell that to Junio. He is also repeating the same.\n\nJunio explained you carefully how the former code was purposely aligning\nstuff and you broke it.\n\n> I actually provided *evidence*. Did you miss it?\n\nStar-emphasis does not make things true either. Again, you're losing\nyour time, and won't lose mine any further.\n\n-- \nMatthieu Moy\nhttp://www-verimag.imag.fr/~moy/\n"},{"id":"217735","messageId":"CAMP44s1uDEUxxJWwHi=J1Jf=CnApi99BW3ZxTPUa7T+py==gfQ@mail.gmail.com","threadId":"33756","inReplyTo":"vpqwqqxbka7.fsf@grenoble-inp.fr","subject":"Re: [PATCH 3/4] {fast-export,transport-helper}: style cleanups","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-17T17:18:32Z","receivedAt":"2013-05-17T17:18:32Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Fri, May 17, 2013 at 12:14 PM, Matthieu Moy\n<Matthieu.Moy@grenoble-inp.fr> wrote:\n> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>\n>> On Fri, May 17, 2013 at 11:56 AM, Matthieu Moy\n>> <Matthieu.Moy@grenoble-inp.fr> wrote:\n>>> Felipe Contreras <felipe.contreras@gmail.com> writes:\n>>>\n>>>> THERE IS NO STYLE BREAKAGE.\n>>>\n>>> Repeating something, and even making it all caps does not make it true.\n>>\n>> Tell that to Junio. He is also repeating the same.\n>\n> Junio explained you carefully how the former code was purposely aligning\n> stuff and you broke it.\n\nYeah, and it was purposely adding a space between the parenthesis,\nthat doesn't mean it's the desired style.\n\nThe fact that something is a certain way, or was, doesn't imply it\n*should* be that way. Explaining what the code was formerly doing is\nno argument, it's merely an observation.\n\n>> I actually provided *evidence*. Did you miss it?\n>\n> Star-emphasis does not make things true either. Again, you're losing\n> your time, and won't lose mine any further.\n\nOf course, because you have no evidence or arguments.\n\n-- \nFelipe Contreras\n"}]}