Re: Request for comment on setting binary format output per session

On Tue, 18 Apr 2023 at 12:24, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:

> On Tue, Apr 18, 2023 at 11:51 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> > > One thing I think we should do in this area is introduce #defines for
> > > all the message type codes and use those instead of having hard-coded
> > > constants everywhere.
> >
> > +1, but I wonder where we should put those exactly. My first thought
> > was postgres_ext.h, but the charter for that is
> >
> > * This file contains declarations of things that are visible
> everywhere
> > * in PostgreSQL *and* are visible to clients of frontend interface
> libraries.
> > * For example, the Oid type is part of the API of libpq and other
> libraries.
> >
> > so picayune details of the wire protocol probably don't belong there.
> > Maybe we need a new header concerned with the wire protocol?
>
> Yeah. I sort of thought maybe one of the files in src/include/libpq
> would be the right place, but it doesn't look like it.
>
> If we at least created the defines and replaced occurrences with the same,
then we can litigate where to put them later.

I think I'd prefer this in a different patch, but I'd be willing to take a
run at it.

Dave