On Thu, 5 Jul 2018 19:08:58 +0300 "Yordan Karadzhov (VMware)" <y.karadz@xxxxxxxxx> wrote: > Hi Steve, > I am still learning how to answer to patches, and I guess because of > this you didn't notice my comment at the bottom. I think you have to > remove the line > > free_record(rec); My fault, I should have looked further down. This is why I usually clean up my replies. By removing functions I'm not commenting on and replacing them with: [..] Even with that said, it was my fault for missing it. Thanks for the review. > > You don't have to send me another patch. OK, I'll fix it up and push it out. -- Steve