> On 05-Aug-2020, at 7:01 PM, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote:
> 
> On 2020-Aug-05, Asim Praveen wrote:
> 
>> Please find attached a StringInfo based solution to this problem.  It
>> uses fgetln instead of fgets such that a line is read in full, without
>> ever splitting it.
> 
> never heard of fgetln, my system doesn't have a manpage for it, and we
> don't use it anywhere AFAICS.  Are you planning to add something to
> src/common for it?
> 

Indeed!  I noticed fgetln on the man page of fgets and used it without 
checking.  And this happened on a MacOS system.

Please find a revised version that uses fgetc instead.

Asim

Attachment: v2-0001-Use-a-stringInfo-instead-of-a-char-for-replace_st.patch
Description: v2-0001-Use-a-stringInfo-instead-of-a-char-for-replace_st.patch

Reply via email to