From: Heikki Linnakangas <hlinnaka@iki.fi>
To: Daniel Gustafsson <daniel@yesql.se>
To: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
Cc: robertmhaas@gmail.com
Cc: tsunakawa.takay@jp.fujitsu.com
Cc: tgl@sss.pgh.pa.us
Cc: ishii@sraoss.co.jp
Cc: pgsql-hackers@postgresql.org
Subject: Re: Radix tree for character conversion
Date: Wed, 30 Nov 2016 14:59:44 +0200
Message-ID: <ca267403-9987-7abb-c98e-fb4a8ac2d76a@iki.fi> (raw)
In-Reply-To: <3FC648B5-2B7F-4585-9615-207A44B730A9@yesql.se>
References: <6d85d710-9554-a928-29ff-b2d3b80b01c9@iki.fi>
<20161021.173321.105120238.horiguchi.kyotaro@lab.ntt.co.jp>
<08e7892a-d55c-eefe-76e6-7910bc8dd1f3@iki.fi>
<20161027.162337.204351475.horiguchi.kyotaro@lab.ntt.co.jp>
<3FC648B5-2B7F-4585-9615-207A44B730A9@yesql.se>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
On 10/31/2016 06:11 PM, Daniel Gustafsson wrote:
>> On 27 Oct 2016, at 09:23, Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> wrote:
>>
>> At Tue, 25 Oct 2016 12:23:48 +0300, Heikki Linnakangas <hlinnaka@iki.fi> wrote in <08e7892a-d55c-eefe-76e6-7910bc8dd1f3@iki.fi>
>>>
>>> [..]
>>> The perl scripts are still quite messy. For example, I lost the checks
>>> for duplicate mappings somewhere along the way - that ought to be put
>>> back. My Perl skills are limited.
>>
>> Perl scripts are to be messy, I believe. Anyway the duplicate
>> check as been built into the sub print_radix_trees. Maybe the
>> same check is needed by some plain map files but it would be just
>> duplication for the maps having radix tree.
>
> I took a small stab at doing some cleaning of the Perl scripts, mainly around
> using the more modern (well, modern as in +15 years old) form for open(..),
> avoiding global filehandles for passing scalar references and enforcing use
> strict. Some smaller typos and fixes were also included. It seems my Perl has
> become a bit rusty so I hope the changes make sense. The produced files are
> identical with these patches applied, they are merely doing cleaning as opposed
> to bugfixing.
>
> The attached patches are against the 0001-0006 patches from Heikki and you in
> this series of emails, the separation is intended to make them easier to read.
Thanks! Patches 0001-0003 seem to have been mostly unchanged for the
later discussion and everyone seems to be happy with those patches, so I
picked the parts of these cleanups of yours that applied to my patches
0001-0003, and pushed those. I'll continue reviewing the rest..
- Heikki
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: hlinnaka@iki.fi, daniel@yesql.se, horiguchi.kyotaro@lab.ntt.co.jp, robertmhaas@gmail.com, tsunakawa.takay@jp.fujitsu.com, tgl@sss.pgh.pa.us, ishii@sraoss.co.jp
Subject: Re: Radix tree for character conversion
In-Reply-To: <ca267403-9987-7abb-c98e-fb4a8ac2d76a@iki.fi>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox