On Fri, Feb 24, 2023 at 4:10 PM Jonathan Tan <jonathantanmy@xxxxxxxxxx> wrote: > > "Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > > This series cleans up headers a bit, trying to remove excessive dependency > > on "cache.h". I created this series a while ago, but decided to clean it up > > and submit it due to Emily's recent thread and suggestion that this might be > > helpful to their efforts[1]. > > [snip] > > > [1] Search for "Extremely yes" in > > https://lore.kernel.org/git/CAJoAoZm+TkCL0Jpg_qFgKottxbtiG2QOiY0qGrz3-uQy+=waPg@xxxxxxxxxxxxxx/ > > Thanks, the series looks good. Thanks for the reviews! > I especially appreciated the separating > out of alloc.h and moving function declarations to the .h of the same > name as the .c file their definition is found in. There are several more cases where we could do this, but my series was already long enough and I wanted to get it reviewed (and make sure others agreed with the directions) before adding more. I'm glad you like this direction, since I'd really like to move more things this way.
