{"thread":{"id":"38193","subject":"[PATCH 1/2] Documentation/SubmittingPatches: Explain the rationale of git notes","startedAt":"2014-12-18T01:08:13Z","lastAt":"2014-12-30T16:19:36Z","messageCount":10,"participants":["Stefan Beller","Junio C Hamano","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"253794","messageId":"1418864895-18583-1-git-send-email-sbeller@google.com","threadId":"38193","inReplyTo":"xmqq1tnx961q.fsf@gitster.dls.corp.google.com","subject":"[PATCH 1/2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-18T01:08:13Z","receivedAt":"2014-12-18T01:08:13Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"This adds an explanation of why you want to have the --notes option\ngiven to git format-patch.\n\nSigned-off-by: Stefan Beller <sbeller@google.com>\n---\n\nNotes:\n    > with optionally update Documentation/SubmittingPatches\n    > to point at the file.\n    \n    That file actually talks about notes already.  I am sending\n    two patches touching that file, one of them explaining\n    the --notes workflow rationale and one of them just changing\n    white spaces.\n    \n    A few weeks ago I wanted to patch format-patch to remove\n    change ids. This is not needed any more for the git workflow\n    as I disabled them and do not upload any patches to an optional\n    Gerrit code review server anymore.\n    \n    I do like the workflow using --notes as well from a developers\n    perspective as I take literally notes for my own sanity.\n    I wonder if I should add a config format.notes = [default-off,\n    always, on-if-non-empty] so I don't need always add --notes\n    manually to the command line.\n    \n    Thanks,\n    Stefan\n\n Documentation/SubmittingPatches | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\nindex fa71b5f..16b5d65 100644\n--- a/Documentation/SubmittingPatches\n+++ b/Documentation/SubmittingPatches\n@@ -176,7 +176,11 @@ message starts, you can put a \"From: \" line to name that person.\n You often want to add additional explanation about the patch,\n other than the commit message itself.  Place such \"cover letter\"\n material between the three dash lines and the diffstat. Git-notes\n-can also be inserted using the `--notes` option.\n+can also be inserted using the `--notes` option. If you are one\n+of those developers who cannot write perfect code the first time\n+and need multiple iterations of review and discussion, you are\n+encouraged to use the notes to describe the changes between the\n+different versions of a patch.\n \n Do not attach the patch as a MIME attachment, compressed or not.\n Do not let your e-mail client send quoted-printable.  Do not let\n-- \n2.2.0.31.gad78000.dirty\n"},{"id":"253795","messageId":"1418864895-18583-2-git-send-email-sbeller@google.com","threadId":"38193","inReplyTo":"1418864895-18583-1-git-send-email-sbeller@google.com","subject":"[PATCH] Documentation/SubmittingPatches: unify whitespace/tabs for the DCO","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-18T01:08:14Z","receivedAt":"2014-12-18T01:08:14Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"The Developers Certificate of Origin has a mixture of tabs and white\nspaces which is annoying to view if your editor explicitly views white\nspace characters.\n\nSigned-off-by: Stefan Beller <sbeller@google.com>\n---\n Documentation/SubmittingPatches | 12 ++++++------\n 1 file changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\nindex 16b5d65..6124f34 100644\n--- a/Documentation/SubmittingPatches\n+++ b/Documentation/SubmittingPatches\n@@ -258,15 +258,15 @@ pretty simple: if you can certify the below:\n             person who certified (a), (b) or (c) and I have not modified\n             it.\n \n-\t(d) I understand and agree that this project and the contribution\n-\t    are public and that a record of the contribution (including all\n-\t    personal information I submit with it, including my sign-off) is\n-\t    maintained indefinitely and may be redistributed consistent with\n-\t    this project or the open source license(s) involved.\n+        (d) I understand and agree that this project and the contribution\n+            are public and that a record of the contribution (including all\n+            personal information I submit with it, including my sign-off) is\n+            maintained indefinitely and may be redistributed consistent with\n+            this project or the open source license(s) involved.\n \n then you just add a line saying\n \n-\tSigned-off-by: Random J Developer <random@developer.example.org>\n+        Signed-off-by: Random J Developer <random@developer.example.org>\n \n This line can be automatically added by Git if you run the git-commit\n command with the -s option.\n-- \n2.2.0.31.gad78000.dirty\n"},{"id":"253796","messageId":"1418864895-18583-3-git-send-email-sbeller@google.com","threadId":"38193","inReplyTo":"1418864895-18583-1-git-send-email-sbeller@google.com","subject":"[PATCH 2/2] Documentation/SubmittingPatches: unify whitespace/tabs for the DCO","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-18T01:08:15Z","receivedAt":"2014-12-18T01:08:15Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"The Developers Certificate of Origin has a mixture of tabs and white\nspaces which is annoying to view if your editor explicitly views white\nspace characters.\n\nSigned-off-by: Stefan Beller <sbeller@google.com>\n---\n\nNotes:\n    I'll also try to send it upstream to linux.\n\n Documentation/SubmittingPatches | 12 ++++++------\n 1 file changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\nindex 16b5d65..6124f34 100644\n--- a/Documentation/SubmittingPatches\n+++ b/Documentation/SubmittingPatches\n@@ -258,15 +258,15 @@ pretty simple: if you can certify the below:\n             person who certified (a), (b) or (c) and I have not modified\n             it.\n \n-\t(d) I understand and agree that this project and the contribution\n-\t    are public and that a record of the contribution (including all\n-\t    personal information I submit with it, including my sign-off) is\n-\t    maintained indefinitely and may be redistributed consistent with\n-\t    this project or the open source license(s) involved.\n+        (d) I understand and agree that this project and the contribution\n+            are public and that a record of the contribution (including all\n+            personal information I submit with it, including my sign-off) is\n+            maintained indefinitely and may be redistributed consistent with\n+            this project or the open source license(s) involved.\n \n then you just add a line saying\n \n-\tSigned-off-by: Random J Developer <random@developer.example.org>\n+        Signed-off-by: Random J Developer <random@developer.example.org>\n \n This line can be automatically added by Git if you run the git-commit\n command with the -s option.\n-- \n2.2.0.31.gad78000.dirty\n"},{"id":"253932","messageId":"xmqqwq5ja7s2.fsf@gitster.dls.corp.google.com","threadId":"38193","inReplyTo":"1418864895-18583-1-git-send-email-sbeller@google.com","subject":"Re: [PATCH 1/2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-12-22T17:55:41Z","receivedAt":"2014-12-22T17:55:41Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller <sbeller@google.com> writes:\n\n> This adds an explanation of why you want to have the --notes option\n> given to git format-patch.\n>\n> Signed-off-by: Stefan Beller <sbeller@google.com>\n> ---\n>\n> Notes:\n>     > with optionally update Documentation/SubmittingPatches\n>     > to point at the file.\n>     \n>     That file actually talks about notes already.  I am sending\n>     two patches touching that file, one of them explaining\n>     the --notes workflow rationale and one of them just changing\n>     white spaces.\n>     \n>     A few weeks ago I wanted to patch format-patch to remove\n>     change ids. This is not needed any more for the git workflow\n>     as I disabled them and do not upload any patches to an optional\n>     Gerrit code review server anymore.\n>     \n>     I do like the workflow using --notes as well from a developers\n>     perspective as I take literally notes for my own sanity.\n>     I wonder if I should add a config format.notes = [default-off,\n>     always, on-if-non-empty] so I don't need always add --notes\n>     manually to the command line.\n>     \n>     Thanks,\n>     Stefan\n>\n>  Documentation/SubmittingPatches | 6 +++++-\n>  1 file changed, 5 insertions(+), 1 deletion(-)\n>\n> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\n> index fa71b5f..16b5d65 100644\n> --- a/Documentation/SubmittingPatches\n> +++ b/Documentation/SubmittingPatches\n> @@ -176,7 +176,11 @@ message starts, you can put a \"From: \" line to name that person.\n>  You often want to add additional explanation about the patch,\n>  other than the commit message itself.  Place such \"cover letter\"\n>  material between the three dash lines and the diffstat. Git-notes\n> -can also be inserted using the `--notes` option.\n> +can also be inserted using the `--notes` option. If you are one\n\nBecause the previous sentence is talking about the cover letter to\ndescribe the overall series, it probably is a good idea to add\n\"after the three-dashes of each patch\" after \"... using the notes\noption\" for clarity, perhaps?\n\n> +of those developers who cannot write perfect code the first time\n> +and need multiple iterations of review and discussion, you are\n> +encouraged to use the notes to describe the changes between the\n> +different versions of a patch.\n\nPerhaps s/you are encouraged to/you may want to/?  It might be\nbetter to be even more explicit, e.g. \"you may want to keep track of\nthe changes between the versions using the notes and then use\n`--notes` when preparing the series for submission\"?\n\n>  Do not attach the patch as a MIME attachment, compressed or not.\n>  Do not let your e-mail client send quoted-printable.  Do not let\n\nThanks.\n"},{"id":"253933","messageId":"xmqqsig7a7rk.fsf@gitster.dls.corp.google.com","threadId":"38193","inReplyTo":"1418864895-18583-3-git-send-email-sbeller@google.com","subject":"Re: [PATCH 2/2] Documentation/SubmittingPatches: unify whitespace/tabs for the DCO","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-12-22T17:55:59Z","receivedAt":"2014-12-22T17:55:59Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller <sbeller@google.com> writes:\n\n> The Developers Certificate of Origin has a mixture of tabs and white\n> spaces which is annoying to view if your editor explicitly views white\n> space characters.\n>\n> Signed-off-by: Stefan Beller <sbeller@google.com>\n> ---\n>\n> Notes:\n>     I'll also try to send it upstream to linux.\n\nThanks.\n\n>\n>  Documentation/SubmittingPatches | 12 ++++++------\n>  1 file changed, 6 insertions(+), 6 deletions(-)\n>\n> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\n> index 16b5d65..6124f34 100644\n> --- a/Documentation/SubmittingPatches\n> +++ b/Documentation/SubmittingPatches\n> @@ -258,15 +258,15 @@ pretty simple: if you can certify the below:\n>              person who certified (a), (b) or (c) and I have not modified\n>              it.\n>  \n> -\t(d) I understand and agree that this project and the contribution\n> -\t    are public and that a record of the contribution (including all\n> -\t    personal information I submit with it, including my sign-off) is\n> -\t    maintained indefinitely and may be redistributed consistent with\n> -\t    this project or the open source license(s) involved.\n> +        (d) I understand and agree that this project and the contribution\n> +            are public and that a record of the contribution (including all\n> +            personal information I submit with it, including my sign-off) is\n> +            maintained indefinitely and may be redistributed consistent with\n> +            this project or the open source license(s) involved.\n>  \n>  then you just add a line saying\n>  \n> -\tSigned-off-by: Random J Developer <random@developer.example.org>\n> +        Signed-off-by: Random J Developer <random@developer.example.org>\n>  \n>  This line can be automatically added by Git if you run the git-commit\n>  command with the -s option.\n"},{"id":"254140","messageId":"CAGZ79kYuivGwayvJ+D8vOBP4yHN64Cx6UdxrSn430GyJ3K6ywg@mail.gmail.com","threadId":"38193","inReplyTo":"xmqqwq5ja7s2.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH 1/2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-29T17:36:59Z","receivedAt":"2014-12-29T17:36:59Z","isPatch":true,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Mon, Dec 22, 2014 at 9:55 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Stefan Beller <sbeller@google.com> writes:\n>\n>> This adds an explanation of why you want to have the --notes option\n>> given to git format-patch.\n>>\n>> Signed-off-by: Stefan Beller <sbeller@google.com>\n>> ---\n>>\n>> Notes:\n>>     > with optionally update Documentation/SubmittingPatches\n>>     > to point at the file.\n>>\n>>     That file actually talks about notes already.  I am sending\n>>     two patches touching that file, one of them explaining\n>>     the --notes workflow rationale and one of them just changing\n>>     white spaces.\n>>\n>>     A few weeks ago I wanted to patch format-patch to remove\n>>     change ids. This is not needed any more for the git workflow\n>>     as I disabled them and do not upload any patches to an optional\n>>     Gerrit code review server anymore.\n>>\n>>     I do like the workflow using --notes as well from a developers\n>>     perspective as I take literally notes for my own sanity.\n>>     I wonder if I should add a config format.notes = [default-off,\n>>     always, on-if-non-empty] so I don't need always add --notes\n>>     manually to the command line.\n>>\n>>     Thanks,\n>>     Stefan\n>>\n>>  Documentation/SubmittingPatches | 6 +++++-\n>>  1 file changed, 5 insertions(+), 1 deletion(-)\n>>\n>> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\n>> index fa71b5f..16b5d65 100644\n>> --- a/Documentation/SubmittingPatches\n>> +++ b/Documentation/SubmittingPatches\n>> @@ -176,7 +176,11 @@ message starts, you can put a \"From: \" line to name that person.\n>>  You often want to add additional explanation about the patch,\n>>  other than the commit message itself.  Place such \"cover letter\"\n>>  material between the three dash lines and the diffstat. Git-notes\n>> -can also be inserted using the `--notes` option.\n>> +can also be inserted using the `--notes` option. If you are one\n>\n> Because the previous sentence is talking about the cover letter to\n> describe the overall series, it probably is a good idea to add\n> \"after the three-dashes of each patch\" after \"... using the notes\n> option\" for clarity, perhaps?\n\nThe previous sentence is talking about additional information,\nwhich should not go into the commit message. (very similar to\nthe cover letter but just for one patch). It already mentions the place\n\"between the three dash lines and the diffstat.\"\n\n>\n>> +of those developers who cannot write perfect code the first time\n>> +and need multiple iterations of review and discussion, you are\n>> +encouraged to use the notes to describe the changes between the\n>> +different versions of a patch.\n>\n> Perhaps s/you are encouraged to/you may want to/?  It might be\n> better to be even more explicit, e.g. \"you may want to keep track of\n> the changes between the versions using the notes and then use\n> `--notes` when preparing the series for submission\"?\n\nI did reword it to make it more obvious.\n\n>\n>>  Do not attach the patch as a MIME attachment, compressed or not.\n>>  Do not let your e-mail client send quoted-printable.  Do not let\n>\n> Thanks.\n\nA paragraph before that change we have\n\n    \"git format-patch\" command follows the best current practice to\n    format the body of an e-mail message.\n\nwhich makes me wonder if the notes are a best practice or may be\nbecoming a best practice. So maybe we want to default to add the notes\nin format-patch instead of explicitly asking for it. But that's just an idea\nsuited for another patch.\n\nThanks\n"},{"id":"254142","messageId":"1419874942-9901-1-git-send-email-sbeller@google.com","threadId":"38193","inReplyTo":"xmqqwq5ja7s2.fsf@gitster.dls.corp.google.com","subject":"[PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-29T17:42:22Z","receivedAt":"2014-12-29T17:42:22Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"This adds more explanation of why you want to have the --notes option\ngiven to git format-patch.\n\nSigned-off-by: Stefan Beller <sbeller@google.com>\n---\n\nNotes:\n    Changes v2:\n     * s/you are encouraged to/you may want to/\n     * a stronger hint to use the git notes and then\n       --notes for format-patch.\n\n Documentation/SubmittingPatches | 7 ++++++-\n 1 file changed, 6 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\nindex e3c942e..f42c607 100644\n--- a/Documentation/SubmittingPatches\n+++ b/Documentation/SubmittingPatches\n@@ -177,7 +177,12 @@ message starts, you can put a \"From: \" line to name that person.\n You often want to add additional explanation about the patch,\n other than the commit message itself.  Place such \"cover letter\"\n material between the three dash lines and the diffstat. Git-notes\n-can also be inserted using the `--notes` option.\n+can also be inserted using the `--notes` option. If you are one\n+of those developers who cannot write perfect code the first time\n+and need multiple iterations of review and discussion, you may\n+want to keep track of the changes between different versions of\n+a patch using notes and then also use the `--notes` option when\n+preparing the patch for submission.\n \n Do not attach the patch as a MIME attachment, compressed or not.\n Do not let your e-mail client send quoted-printable.  Do not let\n-- \n2.2.1.62.g3f15098\n"},{"id":"254172","messageId":"CAPig+cQgG5Ua-k1D8yKF5K+tfXeSB_wTN9W3m08F+K7AtKWDaw@mail.gmail.com","threadId":"38193","inReplyTo":"1419874942-9901-1-git-send-email-sbeller@google.com","subject":"Re: [PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2014-12-29T23:18:37Z","receivedAt":"2014-12-29T23:18:37Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Mon, Dec 29, 2014 at 12:42 PM, Stefan Beller <sbeller@google.com> wrote:\n> This adds more explanation of why you want to have the --notes option\n> given to git format-patch.\n>\n> Signed-off-by: Stefan Beller <sbeller@google.com>\n> ---\n> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\n> index e3c942e..f42c607 100644\n> --- a/Documentation/SubmittingPatches\n> +++ b/Documentation/SubmittingPatches\n> @@ -177,7 +177,12 @@ message starts, you can put a \"From: \" line to name that person.\n>  You often want to add additional explanation about the patch,\n>  other than the commit message itself.  Place such \"cover letter\"\n>  material between the three dash lines and the diffstat. Git-notes\n> -can also be inserted using the `--notes` option.\n> +can also be inserted using the `--notes` option. If you are one\n> +of those developers who cannot write perfect code the first time\n> +and need multiple iterations of review and discussion, you may\n> +want to keep track of the changes between different versions of\n> +a patch using notes and then also use the `--notes` option when\n> +preparing the patch for submission.\n\nPerhaps this could be rephrased in a less derogatory fashion like this:\n\n    ...material between the three dash line and the diffstat.\n    For patches requiring multiple iterations of review and\n    discussion, an explanation of changes between each iteration can\n    be kept in Git-notes and inserted automatically following the\n    three dash line via `git format-patch --notes`.\n\n>  Do not attach the patch as a MIME attachment, compressed or not.\n>  Do not let your e-mail client send quoted-printable.  Do not let\n> --\n> 2.2.1.62.g3f15098\n"},{"id":"254173","messageId":"CAGZ79kZxsWZRdaQ-0_9Ou3y0KOruD0WVrC0Drkjgu=TbgUjiSA@mail.gmail.com","threadId":"38193","inReplyTo":"CAPig+cQgG5Ua-k1D8yKF5K+tfXeSB_wTN9W3m08F+K7AtKWDaw@mail.gmail.com","subject":"Re: [PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2014-12-29T23:21:41Z","receivedAt":"2014-12-29T23:21:41Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Mon, Dec 29, 2014 at 3:18 PM, Eric Sunshine <sunshine@sunshineco.com> wrote:\n> On Mon, Dec 29, 2014 at 12:42 PM, Stefan Beller <sbeller@google.com> wrote:\n>> This adds more explanation of why you want to have the --notes option\n>> given to git format-patch.\n>>\n>> Signed-off-by: Stefan Beller <sbeller@google.com>\n>> ---\n>> diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\n>> index e3c942e..f42c607 100644\n>> --- a/Documentation/SubmittingPatches\n>> +++ b/Documentation/SubmittingPatches\n>> @@ -177,7 +177,12 @@ message starts, you can put a \"From: \" line to name that person.\n>>  You often want to add additional explanation about the patch,\n>>  other than the commit message itself.  Place such \"cover letter\"\n>>  material between the three dash lines and the diffstat. Git-notes\n>> -can also be inserted using the `--notes` option.\n>> +can also be inserted using the `--notes` option. If you are one\n>> +of those developers who cannot write perfect code the first time\n>> +and need multiple iterations of review and discussion, you may\n>> +want to keep track of the changes between different versions of\n>> +a patch using notes and then also use the `--notes` option when\n>> +preparing the patch for submission.\n>\n> Perhaps this could be rephrased in a less derogatory fashion like this:\n\nThis wasn't meant to be derogatory at all. It was more of subtle humor.\nSorry for the confusion.\n\nIf this is offensive in any way I'd rather go with your suggestion indeed.\n\n>\n>     ...material between the three dash line and the diffstat.\n>     For patches requiring multiple iterations of review and\n>     discussion, an explanation of changes between each iteration can\n>     be kept in Git-notes and inserted automatically following the\n>     three dash line via `git format-patch --notes`.\n>\n>>  Do not attach the patch as a MIME attachment, compressed or not.\n>>  Do not let your e-mail client send quoted-printable.  Do not let\n>> --\n>> 2.2.1.62.g3f15098\n"},{"id":"254200","messageId":"xmqqzja5rtyf.fsf@gitster.dls.corp.google.com","threadId":"38193","inReplyTo":"CAGZ79kZxsWZRdaQ-0_9Ou3y0KOruD0WVrC0Drkjgu=TbgUjiSA@mail.gmail.com","subject":"Re: [PATCHv2] Documentation/SubmittingPatches: Explain the rationale of git notes","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-12-30T16:19:36Z","receivedAt":"2014-12-30T16:19:36Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller <sbeller@google.com> writes:\n\n>>> +can also be inserted using the `--notes` option. If you are one\n>>> +of those developers who cannot write perfect code the first time\n>>> +and need multiple iterations of review and discussion, you may\n>>> +want to keep track of the changes between different versions of\n>>> +a patch using notes and then also use the `--notes` option when\n>>> +preparing the patch for submission.\n>>\n>> Perhaps this could be rephrased in a less derogatory fashion like this:\n>\n> This wasn't meant to be derogatory at all. It was more of subtle humor.\n> Sorry for the confusion.\n>\n> If this is offensive in any way I'd rather go with your suggestion indeed.\n\nI too found it very close to the borderline between being funny and\nbeing unnecessarily condescending when I first read it and couldn't\ntell which side of the line it was on.\n\nIt is not if the readers need multiple iterations to get things\nperfect (most of us do, anyway).  It is if the readers already have\na good procedure to manage the changes (if they do, they do not need\nus to making suggestion to use notes) or they are actively in search\nof a good tool to add to their toolbox.  From that point of view,\nEric's rewrite does present a useful tool to keep track of changes\nthat the readers may want to consider in a lot tamer way.\n"}]}