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
