{"thread":{"id":"45193","subject":"[PATCH] Documentation: use brackets for optional arguments","startedAt":"2017-02-22T12:26:00Z","lastAt":"2017-02-22T18:22:01Z","messageCount":2,"participants":["brian m. carlson","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"312278","messageId":"20170222122546.922199-1-sandals@crustytoothpaste.net","threadId":"45193","inReplyTo":null,"subject":"[PATCH] Documentation: use brackets for optional arguments","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2017-02-22T12:25:46Z","receivedAt":"2017-02-22T12:26:00Z","isPatch":true,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"The documentation for git blame used vertical bars for optional\narguments to -M and -C, which is unusual and potentially confusing.\nSince most man pages use brackets for optional items, and that's\nconsistent with how we document the same options for git diff and\nfriends, use brackets here, too.\n\nSigned-off-by: brian m. carlson <sandals@crustytoothpaste.net>\n---\n Documentation/blame-options.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\nindex 2669b87c9d..dc41957afa 100644\n--- a/Documentation/blame-options.txt\n+++ b/Documentation/blame-options.txt\n@@ -77,7 +77,7 @@ include::line-range-format.txt[]\n \tterminal. Can't use `--progress` together with `--porcelain`\n \tor `--incremental`.\n \n--M|<num>|::\n+-M[<num>]::\n \tDetect moved or copied lines within a file. When a commit\n \tmoves or copies a block of lines (e.g. the original file\n \thas A and then B, and the commit changes it to B and then\n@@ -93,7 +93,7 @@ alphanumeric characters that Git must detect as moving/copying\n within a file for it to associate those lines with the parent\n commit. The default value is 20.\n \n--C|<num>|::\n+-C[<num>]::\n \tIn addition to `-M`, detect lines moved or copied from other\n \tfiles that were modified in the same commit.  This is\n \tuseful when you reorganize your program and move code\n"},{"id":"312327","messageId":"xmqq37f6cchp.fsf@gitster.mtv.corp.google.com","threadId":"45193","inReplyTo":"20170222122546.922199-1-sandals@crustytoothpaste.net","subject":"Re: [PATCH] Documentation: use brackets for optional arguments","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-22T18:21:54Z","receivedAt":"2017-02-22T18:22:01Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"brian m. carlson\" <sandals@crustytoothpaste.net> writes:\n\n> The documentation for git blame used vertical bars for optional\n> arguments to -M and -C, which is unusual and potentially confusing.\n> Since most man pages use brackets for optional items, and that's\n> consistent with how we document the same options for git diff and\n> friends, use brackets here, too.\n\nThis seems to date back to April 2007 X-<.\n\nThanks.\n\n>\n> Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>\n> ---\n>  Documentation/blame-options.txt | 4 ++--\n>  1 file changed, 2 insertions(+), 2 deletions(-)\n>\n> diff --git a/Documentation/blame-options.txt b/Documentation/blame-options.txt\n> index 2669b87c9d..dc41957afa 100644\n> --- a/Documentation/blame-options.txt\n> +++ b/Documentation/blame-options.txt\n> @@ -77,7 +77,7 @@ include::line-range-format.txt[]\n>  \tterminal. Can't use `--progress` together with `--porcelain`\n>  \tor `--incremental`.\n>  \n> --M|<num>|::\n> +-M[<num>]::\n>  \tDetect moved or copied lines within a file. When a commit\n>  \tmoves or copies a block of lines (e.g. the original file\n>  \thas A and then B, and the commit changes it to B and then\n> @@ -93,7 +93,7 @@ alphanumeric characters that Git must detect as moving/copying\n>  within a file for it to associate those lines with the parent\n>  commit. The default value is 20.\n>  \n> --C|<num>|::\n> +-C[<num>]::\n>  \tIn addition to `-M`, detect lines moved or copied from other\n>  \tfiles that were modified in the same commit.  This is\n>  \tuseful when you reorganize your program and move code\n"}]}