On 25/03/31 10:41AM, Patrick Steinhardt wrote: > The code that implements block sources is distributed across a couple of > files even though. Consolidate all of it into "reftable/blocksource.c" even though... ? I think you meant to followup with something here. > and its accompanying header so that it is easier to locate and more self > contained. It looks like some of the block source function get renamed in the process. It might we worth mentioning that in the commit message too. Other than that, everything else in this patch seems to be a simple reorganization. Looks good! -Justin