From: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> |
---|---|
To: | Hamid Akhtar <hamid(dot)akhtar(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
Subject: | Re: Minor issues in .pgpass |
Date: | 2020-03-04 10:04:07 |
Message-ID: | 905d0162-c9a6-3261-e16e-bec7a3e336b0@oss.nttdata.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2020/03/03 22:07, Hamid Akhtar wrote:
> On Tue, Mar 3, 2020 at 5:38 PM Hamid Akhtar <hamid(dot)akhtar(at)gmail(dot)com <mailto:hamid(dot)akhtar(at)gmail(dot)com>> wrote:
>
>
>
> On Mon, Mar 2, 2020 at 6:07 PM Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com <mailto:masao(dot)fujii(at)oss(dot)nttdata(dot)com>> wrote:
>
>
>
> On 2020/02/29 0:46, Hamid Akhtar wrote:
> > The following review has been posted through the commitfest application:
> > make installcheck-world: not tested
> > Implements feature: not tested
> > Spec compliant: not tested
> > Documentation: not tested
> >
> > First of all, this seems like fixing a valid issue, albeit, the probability of somebody messing is low, but it is still better to fix this problem.
> >
> > I've not tested the patch in any detail, however, there are a couple of comments I have before I proceed on with detailed testing.
>
> Thanks for the review and comments!
>
> > 1. pgindent is showing a few issues with formatting. Please have a look and resolve those.
>
> Yes.
Fixed. Attached is the updated version of the patch.
I marked this CF entry as "Needs Review" again.
> > 2. I think you can potentially use "len" variable instead of introducing "buflen" and "tmplen" variables.
>
> Basically I don't want to use the same variable for several purposes
> because which would decrease the code readability.
>
>
> That is fine.
>
>
> > Also, I would choose a more appropriate name for "tmp" variable.
>
> Yeah, so what about "rest" as the variable name?
>
>
> May be something like "excess_buf" or any other one that describes that these bytes are to be discarded.
Thanks for the comment! But IMO that "rest" is not
so bad choice, so for now I used "rest" in the latest patch.
Regards,
--
Fujii Masao
NTT DATA CORPORATION
Advanced Platform Technology Group
Research and Development Headquarters
Attachment | Content-Type | Size |
---|---|---|
pgpass_v2.patch | text/plain | 1.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2020-03-04 10:14:36 | Re: Identifying user-created objects |
Previous Message | Fujii Masao | 2020-03-04 09:57:00 | Re: Identifying user-created objects |