[PATCH 0/9] reftable: code style improvements

Hi,

this patch series contains a more or less random assortments of code style improvements for the reftable library. The patch series is structured as follows:

  - Patches 1-2 introduce macros to allocate and grow arrays. These are
    basically the same as our ALLOC_GROW and ALLOC_ARRAY macros, but
    specific to the reftable library.

  - Patches 3-6 refactor some code to use `size_t` to index into array
    slices instead of `int`.

  - Patches 7-9 are some more debatable cleanups that helped me
    personally to make sense of the code better.

I've been a bit hesitant to send out these patches as they may be considered "noise", especially the last three ones. But the reftable library feels quite alien in our codebase, which does increase the mental overhead at least for me when reading its ccode.

I think it should be a goal of ours to align it closer to our normal coding style, which will of course end up in quite a bit of churn over time. A different approach would be to do these refactorings while already touching the code anyway due to other reasons. But I found myself doing the same refactorings over and over again in different contexts for patch series I ultimately didn't end up sending, so I'd really rather want to get those out of my way.

Anyway, please let me know how you feel about patch series like these.

Patrick

Patrick Steinhardt (9):
  reftable: introduce macros to grow arrays
  reftable: introduce macros to allocate arrays
  reftable/stack: fix parameter validation when compacting range
  reftable/stack: index segments with `size_t`
  reftable/stack: use `size_t` to track stack slices during compaction
  reftable/stack: use `size_t` to track stack length
  reftable/merged: refactor seeking of records
  reftable/merged: refactor initialization of iterators
  reftable/record: improve semantics when initializing records

 reftable/basics.c          |  15 ++--
 reftable/basics.h          |  17 ++++-
 reftable/block.c           |  25 +++---
 reftable/block_test.c      |   2 +-
 reftable/blocksource.c     |   4 +-
 reftable/iter.c            |   3 +-
 reftable/merged.c          | 100 +++++++++++-------------
 reftable/merged_test.c     |  52 ++++++-------
 reftable/pq.c              |   8 +-
 reftable/publicbasics.c    |   3 +-
 reftable/reader.c          |  12 ++-
 reftable/readwrite_test.c  |   8 +-
 reftable/record.c          |  43 +++--------
 reftable/record.h          |  10 +--
 reftable/record_test.c     |   8 +-
 reftable/refname.c         |   4 +-
 reftable/reftable-merged.h |   2 +-
 reftable/stack.c           | 151 +++++++++++++++++--------------------
 reftable/stack.h           |   6 +-
 reftable/stack_test.c      |   7 +-
 reftable/tree.c            |   4 +-
 reftable/writer.c          |  21 ++----
 22 files changed, 221 insertions(+), 284 deletions(-)

base-commit: bc7ee2e5e16f0d1e710ef8fab3db59ab11f2bbe7