On 2019-Feb-01, Dmitry Dolgov wrote: > > On Fri, Feb 1, 2019 at 12:33 PM Alvaro Herrera <alvhe...@2ndquadrant.com> > > wrote:
> > > * Use NULL as a default value where it was an empty string before (this > > > required few minor changes for some part of the code outside > > > ArchiveEntry) > > > > I would rename the function to sanitize_line() and put those comments there > > (removing them from the callsites), then the new argument I suggest would > > not > > be completely out of place. > > Yes, sounds pretty reasonable for me. Thanks for looking -- pushed. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
