Tom Lane wrote:
> Magnus Hagander <magnus@hagander.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.
Applied.
//Magnus