On Sun, Nov 24, 2019 at 12:15 AM William Lallemand <wlallem...@haproxy.com> wrote: > That's a remain of the previous way of doing this, which was done with an > array of 2 old_ckch, so the previous check was something like: > > > if (!old_ckchs && !old_ckchs) > > When a transaction is created the old_ckchs and the path are set. > So we can check the path OR the old_ckchs, but it's not needed to do both, > since there are both set and free during a transaction at the same time. > > The goal there was to check if a transaction exists, not to check if we free'd > the path before setting it.
ok got it, thanks for the explanation. In that case I will send a cleanup patch shortly, but I let you decide whether it is worth merging it or wait for another rework to integrate it. -- William