| From: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
| Cc: | "Shulgin, Oleksandr" <oleksandr(dot)shulgin(at)zalando(dot)de>, David Steele <david(at)pgmasters(dot)net>, Scott Mead <scottm(at)openscg(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_hba_lookup function to get all matching pg_hba.conf entries |
| Date: | 2016-03-21 07:31:56 |
| Message-ID: | CAJrrPGexBgEaSoq65LiCW5zSmH4XA3wWz7_a_GTq0s8GB4pETg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Mar 21, 2016 at 2:00 PM, Alvaro Herrera
<alvherre(at)2ndquadrant(dot)com> wrote:
> Haribabu Kommi wrote:
>
>> > Check.
>> >
>> > +} lookup_hba_line_context;
>> > ^^^^^ but why TAB here?
>>
>> corrected. I am not sure why pg_indent is adding a tab here.
>
> It's because lookup_hba_line_context is not listed in typedefs.list.
> I suggest adding it and all other new typedefs you add, and rerunning
> pgindent, as the lack of those may affect other places where those names
> appear.
Thanks for the details. I added the new typedef into typedefs.list file.
Updated patch is attached.
Regards,
Hari Babu
Fujitsu Australia
| Attachment | Content-Type | Size |
|---|---|---|
| pg-hba-lookup-21-03-2016_1.patch | application/octet-stream | 31.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Haribabu Kommi | 2016-03-21 07:43:08 | Re: pam auth - add rhost item |
| Previous Message | Michael Paquier | 2016-03-21 06:54:42 | Re: extend pgbench expressions with functions |