Re: [PATCH] Refactor pqformat.{c,h} and protocol.h - Mailing list pgsql-hackers

From Aleksander Alekseev
Subject Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
Date
Msg-id CAJ7c6TMQNuUaJEYKtY-DNwZa-6yNTNUVsFR16yZLtB0iZZKx7A@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Refactor pqformat.{c,h} and protocol.h  (Nathan Bossart <nathandbossart@gmail.com>)
Responses Re: [PATCH] Refactor pqformat.{c,h} and protocol.h
List pgsql-hackers
Hi,

> As discussed elsewhere [0], we can add the leader/worker protocol
> characters to protocol.h, but they should probably go in a separate
> section.  I'd recommend breaking that part out to a separate patch, too.

OK, here is the updated patchset. This time I chose not to include this patch:

> - Patch 3 rearranges the order of the functions in pqformat.{c,h} a
> bit to make the code easier to read.

... since arguably there is not much value in it. Please let me know
if you think it's actually needed.

-- 
Best regards,
Aleksander Alekseev

Attachment

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: Remove dependence on integer wrapping
Next
From: Joe Conway
Date:
Subject: Re: [18] Policy on IMMUTABLE functions and Unicode updates