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. I'm going to suggest to use "git rebase
-i" so that fixes for bugs that earlier patches introduce are applied as
fix-ups in those patches; we don't need or want to see the submitter's
intermediate versions. Ideally, each submitted patch should be free of
such problems, so that we can consider each individual patch in the
series in isolation. Indeed, evidently the cfbot consider things that
way.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services