Re: vd/fix-unaligned-read-index-v4, was Re: What's cooking in git.git (Sep 2022, #08; Tue, 27)
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 28, 2022, 17:41 UTC
- Message-ID
- <xmqqa66jv1kb.fsf@gitster.g>
- In-Reply-To
- <220928.86h70rwhxh.gmgdl@evledraar.gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 20 quoted lines
> On Wed, Sep 28 2022, Junio C Hamano wrote: > >> Jeff King <peff@peff.net> writes: >> >>> ... I have no objection to adding more comments, but I am >>> happy enough without them (like Junio, it may be that I'm overly >>> familiar with how I expect our get_be() functions to handle alignment). >>> ... >>> So it's mostly just a minor annoyance for running the tests; we're >>> probably better not to change any code, even trivially, this late in the >>> release cycle. >> >> Yup. I never planned to merge the topic to 'master'. The finishing >> touch I expected was to help Phillip and friends with a bit of >> explanation in the log message, and then it would be ready to wait >> in 'next' for the next cycle. > > In the interim are we interested in a minimal patch to the specific > scalar test that's finding this under SANITIZE=undefined, as running > un-cleanly will be new in this release?
Yes, it is a new minor annoyance that is better left this late in the cycle.