Re: [PATCH 4/7] reftable: avoid writing empty keys at the block layer
- From
Han-Wen Nienhuys <hanwen@google.com>
- Date
- Jan 17, 2022, 13:10 UTC
- Message-ID
- <CAFQ2z_NLauUqf8FdWbQVxVmqx5xQ11E5t5SwDDyKyLSa9QU8KA@mail.gmail.com>
- In-Reply-To
- <xmqqpmovds5q.fsf@gitster.g>
On Fri, Jan 14, 2022 at 2:26 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 11 quoted lines
> > + ref.refname = ""; > > + ref.value_type = REFTABLE_REF_DELETION; > > + n = block_writer_add(&bw, &rec); > > + EXPECT(n == REFTABLE_API_ERROR); > > + > > The preimage of this hunk has been invalidated by your 9c498398 > (reftable: make reftable_record a tagged union, 2021-12-22). > > I see that the hn/reftable-coverity-fixes topic, which the commit is > a part of, has been expecting a reroll since last year---are you
I sent a reroll on Jan 12. As that has had more scrutiny, it's best if that lands first. This series is less urgent.
> plannning to rebuild that series after landing this series first?
-- Han-Wen Nienhuys - Google Munich I work 80%. Don't expect answers from me on Fridays. -- Google Germany GmbH, Erika-Mann-Strasse 33, 80636 Munich Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Paul Manicle, Halimah DeLaine Prado