Re: git: b12ccd0bb1b3 - main - stand: Remove double words in source code comments
Date: Tue, 18 Apr 2023 17:56:49 UTC
On 4/17/23 11:59 PM, Alexey Dokuchaev wrote: > On Tue, Apr 18, 2023 at 08:12:51AM +0200, Gordon Bergling wrote: >> On Tue, Apr 18, 2023 at 06:31:28AM +0100, Jessica Clarke wrote: >>> On 18 Apr 2023, at 06:15, Gordon Bergling wrote: >> [...] >>>> --- a/stand/i386/boot0/Makefile >>>> +++ b/stand/i386/boot0/Makefile >>>> @@ -36,7 +36,7 @@ BOOT_BOOT0_FLAGS?= 0x8f >>>> # 0xb6 (182d) corresponds to 10 seconds. >>>> BOOT_BOOT0_TICKS?= 0xb6 >>>> >>>> -# The base address that we the boot0 code to to run it. Don't change this >>>> +# The base address that we the boot0 code to run it. Don't change this >>> >>> This was correct and is now incorrect; read as "that we [load] the >>> boot0 code to in order to run it". >> >> I just reverted the change. Sorry, not a native speaker on this side >> and the sentence was a little hard to understand. > > This reminds me of the situation with commit 8ca67bf1273 which jrtc27@ > pointed out was wrong, so I'd say again that comments like this should > be rephrased: these consequent double words make it hard to understand > and just begging for someone to try to "fix" it again in the future. Yes, I'll try to reword it. Gordon, your second change here was correct (the double "value") and you should go ahead and re-do that one. -- John Baldwin