Re: Binary support for pgoutput plugin - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Binary support for pgoutput plugin
Date
Msg-id 51CC789A-38D2-4DD9-886A-7BAFDA5C03DE@yesql.se
Whole thread Raw
In response to Re: Binary support for pgoutput plugin  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Responses Re: Binary support for pgoutput plugin  (Dave Cramer <davecramer@gmail.com>)
List pgsql-hackers
> On 5 Jul 2020, at 23:11, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
>
> On 2020-Jul-05, Daniel Gustafsson wrote:
>
>>> On 2 Jul 2020, at 18:41, Dave Cramer <davecramer@gmail.com> wrote:
>>>
>>> rebased
>>
>> Thanks!  The new version of 0001 patch has a compiler warning due to mixed
>> declarations and code:
>>
>> worker.c: In function ‘slot_store_data’:
>> worker.c:366:5: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
>
> AFAICS this is fixed in 0005.

Yes and no, 0005 fixes one such instance but the one failing the build is
another one in worker.c (the below being from 0008 which in turn change the row
in question from previous patches):

+               int cursor = tupleData->values[remoteattnum]->cursor;

>  I'm going to suggest to use "git rebase -i"

+1

cheers ./daniel


pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: Binary support for pgoutput plugin
Next
From: Tom Lane
Date:
Subject: Re: Can I use extern "C" in an extension so I can use C++?