> > > > CopyReadFromRawBuf as a name for the new function might be misleading > > > as long as we are only using it for binary data. Maybe > > > CopyReadBinaryData is more appropriate? See attached v4 with these > > > and a few other cosmetic changes. > > > > > > > CopyReadBinaryData() looks meaningful. +1. > > Okay, thanks. Let's have a committer take a look at this then? >
I think yes, unless someone has any more points/review comments. Accordingly the status in the commitfest can be changed. With Regards, Bharath Rupireddy. EnterpriseDB: http://www.enterprisedb.com
