| From: | Dave Cramer <davecramer(at)gmail(dot)com> |
|---|---|
| To: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
| Cc: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Petr Jelinek <petr(dot)jelinek(at)2ndquadrant(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Binary support for pgoutput plugin |
| Date: | 2019-11-05 12:16:10 |
| Message-ID: | CADK3HH+DjY+rFDtZKLd-WTL6kat6eSW8D56+L81Tj0YjHJroPg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, 3 Nov 2019 at 21:47, Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> On Thu, Oct 31, 2019 at 3:03 AM Dave Cramer <davecramer(at)gmail(dot)com> wrote:
> > Ok, I've rebased and reverted logicalrep_read_insert
>
> Hi Dave,
>
> From the code style police (actually just from cfbot, which is set up
> to complain about declarations after statements, a bit of C99 we
> aren't ready for):
>
> proto.c:557:6: error: ISO C90 forbids mixed declarations and code
> [-Werror=declaration-after-statement]
> int len = pq_getmsgint(in, 4); /* read length */
> ^
> proto.c:573:6: error: ISO C90 forbids mixed declarations and code
> [-Werror=declaration-after-statement]
> int len = pq_getmsgint(in, 4); /* read length */
> ^
>
Thomas,
Thanks for the review.
See attached
| Attachment | Content-Type | Size |
|---|---|---|
| 0005-Remove-C99-declaration-and-code.patch | application/octet-stream | 1.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dave Cramer | 2019-11-05 12:51:27 | Re: let's make the list of reportable GUCs configurable (was Re: Add %r substitution for psql prompts to show recovery status) |
| Previous Message | Alexey Bashtanov | 2019-11-05 12:07:50 | Re: log bind parameter values on error |