Re: Radix tree for character conversion

From: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
To: daniel(at)yesql(dot)se
Cc: hlinnaka(at)iki(dot)fi, robertmhaas(at)gmail(dot)com, tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com, tgl(at)sss(dot)pgh(dot)pa(dot)us, ishii(at)sraoss(dot)co(dot)jp, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Radix tree for character conversion
Date: 2016-11-08 02:02:58
Message-ID: 20161108.110258.59832499.horiguchi.kyotaro@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello,

At Mon, 7 Nov 2016 17:19:29 +0100, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote in <39E295B9-7391-40B6-911D-FE852E4604BD(at)yesql(dot)se>
> > On 07 Nov 2016, at 12:32, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> >
> >> On 04 Nov 2016, at 08:34, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> wrote:
> >>
> >> I'm not sure how the discussion about this goes, these patches
> >> makes me think about coding style of Perl.
> >
> > Some of this can absolutely be considered style and more or less down to
> > personal preference. I haven’t seen any coding conventions for Perl so I
> > assume it’s down to consensus among the committers.
>
> Actually, scratch that; there is of course a perltidy profile in the pgindent
> directory. I should avoid sending email before coffee..

Hmm. Somehow perl-mode on my Emacs is stirring with
ununderstandable indentation and I manually correct them so it is
highly probable that the style of this patch is not compatible
with the defined style. Anyway it is better that pgindent
generates smaller patch so I'll try it.

regards,

--
Kyotaro Horiguchi
NTT Open Source Software Center

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2016-11-08 02:10:46 Re: Do we need use more meaningful variables to replace 0 in catalog head files?
Previous Message Hao Lee 2016-11-08 01:57:13 Do we need use more meaningful variables to replace 0 in catalog head files?