Tom Lane wrote: > Magnus Hagander <mag...@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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
