On Thu, Dec 24, 2020 at 09:20:27PM -0500, Tom Lane wrote: > Bruce Momjian <[email protected]> writes: > > I now understand the wisdom of your suggestion. Attached is a patch > > that removes hex_decode from ecpg properly, and returns -1 from the > > /common version. > > I'm fairly unimpressed with this. I don't like having fundamentally > different (and 100% undocumented) behaviors between the frontend and > backend versions of the "same" function. > > I think you should leave ecpglib alone. Unifying that little bit > of code isn't worth having to contort the API of the common version.
OK, sure, I can see that point too. -- Bruce Momjian <[email protected]> https://momjian.us EnterpriseDB https://enterprisedb.com The usefulness of a cup is in its emptiness, Bruce Lee
