Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] clarify palloc comment on quote_literal_cstr - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] clarify palloc comment on quote_literal_cstr
Date
Msg-id 3894912.1744090019@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] clarify palloc comment on quote_literal_cstr  (Michael Paquier <michael@paquier.xyz>)
Responses Re: [PATCH] clarify palloc comment on quote_literal_cstr
List pgsql-hackers
Tree view 
Michael Paquier <michael@paquier.xyz> writes:
> Hmm.  I don't think that grouping all these details in the same
> comment block is an improvement in this case compared to the first
> version, where it is clear which part of the calculation applies to
> what.  In short, I'm OK with how things are on HEAD.

+1.  When I saw the patch I was mainly afraid that pgindent would
make a hash of it.  But it didn't, so I'm cool with it as-is.

            regards, tom lane

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [PATCH] clarify palloc comment on quote_literal_cstr
Next
From: Nico Williams
Date:
Subject: Re: pg16 && GSSAPI && Heimdal/Macos