"Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > I wrote this patch series about a year and a half ago, but never submitted > it. I rebased and updated it due to [0]. [snip] > [0] Search for "Extremely yes" in > https://lore.kernel.org/git/CAJoAoZm+TkCL0Jpg_qFgKottxbtiG2QOiY0qGrz3-uQy+=waPg@xxxxxxxxxxxxxx/ I left some minor comments, but otherwise, this looks good. I do share the desire to avoid unnecessary refactoring churn, but demarcation of private and public fields does make code much clearer and can potentially avoid a whole class of bugs, so I would be happy if this patch set was merged in.