Horton

Re: Ordering of header file inclusion

vignesh C <vignesh21@gmail.com>

View thread
From: vignesh C <vignesh21@gmail.com>
To: Kuntal Ghosh <kuntalghosh.2007@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Peter Eisentraut <peter.eisentraut@2ndquadrant.com>, Andres Freund <andres@anarazel.de>, Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2019-11-10T12:00:44Z
Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Make the order of the header file includes consistent.

  2. Make the order of the header file includes consistent in backend modules.

  3. Make the order of the header file includes consistent in non-backend modules.

  4. Make the order of the header file includes consistent in contrib modules.

Attachments

On Fri, Nov 8, 2019 at 2:22 PM Kuntal Ghosh <kuntalghosh.2007@gmail.com> wrote:
>
> On Sat, Nov 2, 2019 at 7:42 AM vignesh C <vignesh21@gmail.com> wrote:
> >
> > >
> > Thanks Amit for committing the changes.
> > I found couple of more inconsistencies,  the attached patch includes
> > the fix for the same.
> >
> Thanks for the patch. It seems you've to rebase the patch as it
> doesn't apply on the latest HEAD. Apart from that, the changes looks
> good to me.
>

Thanks Kuntal for reviewing the patch. I have attached the patch which
has been rebased on the latest HEAD.

Regards,
Vignesh
EnterpriseDB: http://www.enterprisedb.com