Re: [PATCH 10/16] reftable/block: store block pointer in the block iterator

> On 25/03/31 10:41AM, Patrick Steinhardt wrote:
> > The block iterator requires access to a bunch of data from the
> > underlying `reftable_block` that it is iterating over. This data is
> > stored by copying over relevant data into a separate set of variables.
> > This has multiple downsides:
> > 
> >   - We require more storage space than necessary. This is more of a
> >     theoretical issue as we shouldn't ever have many blocks.
> > 
> >   - We have to perform more bookkeeping, and the variable names are
> >     inconsistent across the two data structures. This can lead to some
> >     confusion.
> > 
> >   - The lifetime of the block iterator is tied to the block anyway, but
> >     we hide that a bit by only storing pointers into the block.
> 
> s/into/in/