Re: [PATCH 04/12] sequencer.c: use commit-slab to mark seen commits
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- May 12, 2018, 14:00 UTC
- Message-ID
- <CACsJy8DOsSFDR8zoXogyqvKaFrdnF=sQK6O+uvu47OGBrT2sWQ@mail.gmail.com>
- In-Reply-To
- <xmqq36yxf0qc.fsf@gitster-ct.c.googlers.com>
On Sat, May 12, 2018 at 3:43 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 12 quoted lines
> Jeff King <peff@peff.net> writes: > >> On Sat, May 12, 2018 at 10:00:20AM +0200, Nguyễn Thái Ngọc Duy wrote: >> >>> +define_commit_slab(commit_seen, int); >> >> Yay, this one is nice and simple. :) This is what I had hoped for all >> along with the slab concept. >> >> -Peff > > Does it need to use a full int, not just a byte per commit, though?
True. Is it time to introduce 'bool' type? I did not want to put the 'char' type up there (but now that I think about it I could have put uint8_t too).
-- Duy