Re: Using defines for protocol characters - Mailing list pgsql-hackers

From Dave Cramer
Subject Re: Using defines for protocol characters
Date
Msg-id CADK3HHLtFuq8PDsV0+ryoqpSxnnPVrS752q5ra9QZ-SiLqVUXg@mail.gmail.com
Whole thread Raw
In response to Re: Using defines for protocol characters  (Tatsuo Ishii <ishii@sraoss.co.jp>)
Responses Re: Using defines for protocol characters
Re: Using defines for protocol characters
List pgsql-hackers


On Thu, 3 Aug 2023 at 13:25, Tatsuo Ishii <ishii@sraoss.co.jp> wrote:
> Greetings,
>
> Attached is a patch which introduces a file protocol.h. Instead of using
> the actual characters everywhere in the code this patch names the
> characters and removes the comments beside each usage.

> +#define DESCRIBE_PREPARED           'S'
> +#define DESCRIBE_PORTAL             'P'

You use these for Close message as well. I don't like the idea because
Close is different message from Describe message.

What about adding following for Close too use them instead?

#define CLOSE_PREPARED           'S'
#define CLOSE_PORTAL             'P'

Good catch. 
I recall when writing this it was a bit hacky.
What do you think of PREPARED_SUB_COMMAND   and PORTAL_SUB_COMMAND instead of duplicating them ?

Dave

pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Track Oldest Initialized WAL Buffer Page
Next
From: David Rowley
Date:
Subject: Re: Fix incorrect start up costs for WindowAgg paths (bug #17862)