From: Siddharth Kannan Date: Thu, 16 Feb 2017 19:39:41 GMT Subject: Re: [PATCH 1/4 v4] revision.c: do not update argv with unknown option Message-ID: In-Reply-To: Hey Matthieu, On 16 February 2017 at 23:52, Matthieu Moy wrote: > > Indeed, I misread the patch. The explanation could be a little bit more > "tired-reviewer-proof" by not using a past tone, perhaps > > 1. setup_revision, which is changed to ... Oh, okay! Sorry about the confusion! Yes, I used the past perfect tense to refer to changes that were made in this particular patch! I will change the message in the next version to something that's in present tense. > > -- > Matthieu Moy > http://www-verimag.imag.fr/~moy/ -- Best Regards, - Siddharth.