{"thread":{"id":"30890","subject":"[PATCH] git-add--interactive.perl: Remove two unused variables","startedAt":"2012-06-24T21:37:34Z","lastAt":"2012-06-25T22:41:56Z","messageCount":7,"participants":["Thomas \"Enki\" Badie","Junio C Hamano","Thomas Rast","Thomas Badie"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"194172","messageId":"4FE7889E.2020009@gmail.com","threadId":"30890","inReplyTo":null,"subject":"[PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Thomas \"Enki\" Badie","fromEmail":"thomas.badie@gmail.com","sentAt":"2012-06-24T21:37:34Z","receivedAt":"2012-06-24T21:37:34Z","isPatch":true,"sender":{"key":"thomas.badie@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1281881?v=4"},"body":"The patch 8f0bef6 refactors this script and leaves the `$fh' variable\nunused in `diff_applies' and `patch_update_file'.\n\nSigned-off-by: Thomas Badie \n---\nHello,\nThis is my first patch. It introduces a really minor change, and I\nalso want to be sure the way I submit it is the right way. Thanks :)\n\n git-add--interactive.perl | 2 --\n 1 files changed, 0 insertions(+), 2 deletions(-)\n\ndiff --git a/git-add--interactive.perl b/git-add--interactive.perl\nindex d948aa8..710764a 100755\n--- a/git-add--interactive.perl\n+++ b/git-add--interactive.perl\n@@ -1067,7 +1067,6 @@ EOF\n }\n\n sub diff_applies {\n- my $fh;\n return run_git_apply($patch_mode_flavour{APPLY_CHECK} . '--check',\n map { @{$_->{TEXT}} } @_);\n }\n@@ -1514,7 +1513,6 @@ sub patch_update_file {\n }\n\n if (@result) {\n- my $fh;\n my @patch = reassemble_patch($head->{TEXT}, @result);\n my $apply_routine = $patch_mode_flavour{APPLY};\n &$apply_routine(@patch);\n\n-- \n1.7.5.4\n"},{"id":"194178","messageId":"7vfw9khrrt.fsf@alter.siamese.dyndns.org","threadId":"30890","inReplyTo":"4FE7889E.2020009@gmail.com","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-06-25T05:41:58Z","receivedAt":"2012-06-25T05:41:58Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Thomas \\\"Enki\\\" Badie\" writes:\n\n> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n> unused in `diff_applies' and `patch_update_file'.\n>\n> Signed-off-by: Thomas Badie \n> ---\n> Hello,\n> This is my first patch. It introduces a really minor change, and I\n> also want to be sure the way I submit it is the right way. Thanks :)\n\nThe patch submission is almost perfect, except that:\n\n (1) the patch seems to be whitespace damaged; and\n\n (2) the author of the problematic commit should have been Cc'ed\n (especially when he is still an active participant on the list)\n to give him a chance to Ack it (I'm adding Thomas for this).\n\nThanks. I like that you identified why this is a good thing by\nquoting the problematic change.\n\n> git-add--interactive.perl | 2 --\n> 1 files changed, 0 insertions(+), 2 deletions(-)\n>\n> diff --git a/git-add--interactive.perl b/git-add--interactive.perl\n> index d948aa8..710764a 100755\n> --- a/git-add--interactive.perl\n> +++ b/git-add--interactive.perl\n> @@ -1067,7 +1067,6 @@ EOF\n> }\n>\n> sub diff_applies {\n> - my $fh;\n> return run_git_apply($patch_mode_flavour{APPLY_CHECK} . '--check',\n> map { @{$_->{TEXT}} } @_);\n> }\n> @@ -1514,7 +1513,6 @@ sub patch_update_file {\n> }\n>\n> if (@result) {\n> - my $fh;\n> my @patch = reassemble_patch($head->{TEXT}, @result);\n> my $apply_routine = $patch_mode_flavour{APPLY};\n> &$apply_routine(@patch);\n"},{"id":"194192","messageId":"87txxzu55d.fsf@thomas.inf.ethz.ch","threadId":"30890","inReplyTo":"7vfw9khrrt.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Thomas Rast","fromEmail":"trast@inf.ethz.ch","sentAt":"2012-06-25T09:12:14Z","receivedAt":"2012-06-25T09:12:14Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Junio C Hamano writes:\n\n> \"Thomas \\\"Enki\\\" Badie\" writes:\n>\n>> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n>> unused in `diff_applies' and `patch_update_file'.\n[...]\n> (2) the author of the problematic commit should have been Cc'ed\n> (especially when he is still an active participant on the list)\n> to give him a chance to Ack it (I'm adding Thomas for this).\n\nIndeed, my bad. It's easy to verify from 'git show 8f0bef6' that this\nwas an oversight in my patch.\n\nAcked-by: Thomas Rast \n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"194193","messageId":"CAFjFENo7qLTZHrEtPPBLGzp=h0w69C+7C2zfiJJmqFANmaWAgg@mail.gmail.com","threadId":"30890","inReplyTo":"7vfw9khrrt.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Thomas Badie","fromEmail":"thomas.badie@gmail.com","sentAt":"2012-06-25T10:12:51Z","receivedAt":"2012-06-25T10:12:51Z","isPatch":true,"sender":{"key":"thomas.badie@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1281881?v=4"},"body":"2012/6/25 Junio C Hamano :\n> \"Thomas \\\"Enki\\\" Badie\" writes:\n>\n>> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n>> unused in `diff_applies' and `patch_update_file'.\n>>\n>> Signed-off-by: Thomas Badie \n>> ---\n>> Hello,\n>> This is my first patch. It introduces a really minor change, and I\n>> also want to be sure the way I submit it is the right way. Thanks :)\n>\n> The patch submission is almost perfect, except that:\n>\n>  (1) the patch seems to be whitespace damaged; and\n>\n\nThe only damage I see is a space on the deleted lines which doesn't\nexist in the patch. Is that you're talking about ?\nI need to investigate why it appears...\n\n>  (2) the author of the problematic commit should have been Cc'ed\n>     (especially when he is still an active participant on the list)\n>     to give him a chance to Ack it (I'm adding Thomas for this).\n\nMy bad, I forgot to check if the commiter were still active. I'll check it\nfor the next time.\n\n> Thanks.  I like that you identified why this is a good thing by\n> quoting the problematic change.\n\nThanks :)\n\n>>  git-add--interactive.perl |    2 --\n>>  1 files changed, 0 insertions(+), 2 deletions(-)\n>>\n>> diff --git a/git-add--interactive.perl b/git-add--interactive.perl\n>> index d948aa8..710764a 100755\n>> --- a/git-add--interactive.perl\n>> +++ b/git-add--interactive.perl\n>> @@ -1067,7 +1067,6 @@ EOF\n>>  }\n>>\n>>  sub diff_applies {\n>> -      my $fh;\n>>        return run_git_apply($patch_mode_flavour{APPLY_CHECK} . '--check',\n>>                             map { @{$_->{TEXT}} } @_);\n>>  }\n>> @@ -1514,7 +1513,6 @@ sub patch_update_file {\n>>        }\n>>\n>>        if (@result) {\n>> -              my $fh;\n>>                my @patch = reassemble_patch($head->{TEXT}, @result);\n>>                my $apply_routine = $patch_mode_flavour{APPLY};\n>>                &$apply_routine(@patch);\n\n-- \nThomas \"Enki\" Badie\n"},{"id":"194194","messageId":"CAFjFENqrGreepYXM3DrDiYEsau9-vM-qmf8KvqrNNjdCS-adjg@mail.gmail.com","threadId":"30890","inReplyTo":"87txxzu55d.fsf@thomas.inf.ethz.ch","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Thomas Badie","fromEmail":"thomas.badie@gmail.com","sentAt":"2012-06-25T10:16:15Z","receivedAt":"2012-06-25T10:16:15Z","isPatch":true,"sender":{"key":"thomas.badie@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1281881?v=4"},"body":"2012/6/25 Thomas Rast :\n> Junio C Hamano writes:\n>\n>> \"Thomas \\\"Enki\\\" Badie\" writes:\n>>\n>>> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n>>> unused in `diff_applies' and `patch_update_file'.\n> [...]\n>>  (2) the author of the problematic commit should have been Cc'ed\n>>      (especially when he is still an active participant on the list)\n>>      to give him a chance to Ack it (I'm adding Thomas for this).\n>\n> Indeed, my bad.  It's easy to verify from 'git show 8f0bef6' that this\n> was an oversight in my patch.\n>\n> Acked-by: Thomas Rast \n\nIt happens to everyone^^ Thanks for your answer.\n\nShould I repost a patch V2 or it is okay like this?\nThanks\n\n-- \nThomas \"Enki\" Badie\n"},{"id":"194219","messageId":"7v7guvgtbi.fsf@alter.siamese.dyndns.org","threadId":"30890","inReplyTo":"CAFjFENo7qLTZHrEtPPBLGzp=h0w69C+7C2zfiJJmqFANmaWAgg@mail.gmail.com","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-06-25T18:06:09Z","receivedAt":"2012-06-25T18:06:09Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thomas Badie writes:\n\n> 2012/6/25 Junio C Hamano :\n>> \"Thomas \\\"Enki\\\" Badie\" writes:\n>>\n>>> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n>>> unused in `diff_applies' and `patch_update_file'.\n>>>\n>>> Signed-off-by: Thomas Badie \n>>> ---\n>>> Hello,\n>>> This is my first patch. It introduces a really minor change, and I\n>>> also want to be sure the way I submit it is the right way. Thanks :)\n>>\n>> The patch submission is almost perfect, except that:\n>>\n>> (1) the patch seems to be whitespace damaged; and\n>>\n>\n> The only damage I see is a space on the deleted lines which doesn't\n> exist in the patch. Is that you're talking about ?\n> I need to investigate why it appears...\n\nA typical context line would begin with SP and typically then tab\n(because our code is indented with leading tabs) but your context\nlines all have two SPs in front and indented with more SPs. This is\na symptom of your MUA mangling your patch, and the reason can be seen\nin the \"Content-Type: text/plain; charset=ISO-8859-1; format=flowed\"\nheader.\n"},{"id":"194233","messageId":"4FE8E934.4090604@gmail.com","threadId":"30890","inReplyTo":"7v7guvgtbi.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] git-add--interactive.perl: Remove two unused variables","fromName":"Thomas Badie","fromEmail":"thomas.badie@gmail.com","sentAt":"2012-06-25T22:41:56Z","receivedAt":"2012-06-25T22:41:56Z","isPatch":true,"sender":{"key":"thomas.badie@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1281881?v=4"},"body":"On 25/06/2012 20:06, Junio C Hamano wrote:\n> Thomas Badie writes:\n> \n>> 2012/6/25 Junio C Hamano :\n>>> \"Thomas \\\"Enki\\\" Badie\" writes:\n>>>\n>>>> The patch 8f0bef6 refactors this script and leaves the `$fh' variable\n>>>> unused in `diff_applies' and `patch_update_file'.\n>>>>\n>>>> Signed-off-by: Thomas Badie \n>>>> ---\n>>>> Hello,\n>>>> This is my first patch. It introduces a really minor change, and I\n>>>> also want to be sure the way I submit it is the right way. Thanks :)\n>>>\n>>> The patch submission is almost perfect, except that:\n>>>\n>>> (1) the patch seems to be whitespace damaged; and\n>>>\n>>\n>> The only damage I see is a space on the deleted lines which doesn't\n>> exist in the patch. Is that you're talking about ?\n>> I need to investigate why it appears...\n> \n> A typical context line would begin with SP and typically then tab\n> (because our code is indented with leading tabs) but your context\n> lines all have two SPs in front and indented with more SPs. This is\n> a symptom of your MUA mangling your patch, and the reason can be seen\n> in the \"Content-Type: text/plain; charset=ISO-8859-1; format=flowed\"\n> header.\n> \n\nI follow the first approach in the tutorial in\n`git format-patch --help`, and I hope it works as expected :)\n\nI wonder what is the next step, should I repost the previous patch with\nthe line \"Acked-by: Thomas Rast \" or it is okay\nright now?\n\nThanks a lot for your help on this first patch,\n\n-- \nThomas \"Enki\" Badie\n"}]}