>
> > > 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