On Fri, Mar 13, 2020 at 11:57:14AM +0100, Tim Düsterhus wrote: > Willy, > > Am 13.03.20 um 08:44 schrieb Willy Tarreau: > >> I've added the new `struct connection` member at the end. Please check > >> whether you think that is the appropriate place for it or if it should > >> be moved somewhere else because of holes or caches. > > > > I'm seeing that the struct connection has become huge (160 bytes now) > > I suspected something like that, that's why I specifically mentioned > that piece. > > > But for now I'm OK with taking your patch above. > > > > I'm not seeing anything in git, yet. > > Is the patch okay as-is and you are just looking into the `struct > connection` refactoring before actually taking it or would you like me > to make any changes to it?
I was willing to take it as is but since I had a comment for the last one and I know that many times when reworking a patch we spot things we wish has been done differently in the previous ones, I let you decide what suits you best :-) If you want me to take it now I can. Willy