Magnus Hagander <magnus(at)hagander(dot)net> writes:
> Tom Lane wrote:
>> It sure looks like that's going to try to free new_parsed_lines more
>> than once. Shouldn't clean_hba_list be done just once?
> Yeah, it should be done in the if branch down below. And I think by our
> coding standards (or at least by our conventions), the variable should
> be "ok" and not "OK".
> Unless there are any objections, I can do those cleanups and apply it.
I thought the adjacent comments could do with a bit more wordsmithing,
also, but otherwise that about covers it.
regards, tom lane