Hi, > Here is the corrected patch. I added it to the nearest CF [1].
I played a bit more with the patch. There was an idea to make ReorderBufferTupleBufData an opaque structure known only within reorderbyffer.c but it turned out that replication/logical/decode.c accesses it directly so I abandoned that idea for now. > Alternatively we could convert ReorderBufferTupleBufData macro to an > inlined function. At least it will add some type safety. Here is v3 that implements it too as a separate patch. Apologies for the noise. -- Best regards, Aleksander Alekseev
v3-0001-Remove-unused-fields-in-ReorderBufferTupleBuf.patch
Description: Binary data
v3-0002-Replace-ReorderBufferTupleBufData-macro-with-a-fu.patch
Description: Binary data
