On Thu, May 05, 2022 at 11:12:45AM +0300, Nikolay Borisov wrote: > nit: You are actually removing this member when copying the struct, that's > an independent change (albeit I'd say insignificant). Generally we prefer > such changes to be in separate patches with rationale when the given member > became redundant. This one actually was entirely unused, but yes, this could have been split into another patch.