Re: Radix tree for character conversion
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
Cc: Michael Paquier <michael.paquier@gmail.com>, hlinnaka <hlinnaka@iki.fi>, Daniel Gustafsson <daniel@yesql.se>,
Peter Eisentraut <peter.eisentraut@2ndquadrant.com>, "Tsunakawa, Takayuki" <tsunakawa.takay@jp.fujitsu.com>,
Tom Lane <tgl@sss.pgh.pa.us>, Tatsuo Ishii <ishii@sraoss.co.jp>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2017-02-28T02:30:22Z
Lists: pgsql-hackers
On Mon, Feb 27, 2017 at 2:07 PM, Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> wrote: >> +# make_charmap - convert charset table to charmap hash >> +# with checking duplicate source code >> Maybe this should be "with checking of duplicated source codes". > > Even though I'm not good English writer, 'duplicated codes' looks > as multiple copies of the original 'code' (for me, of > course). And the 'checking' is a (pure) verbal noun (means not a > deverbal noun) so 'of' is not required. But, of course, I'm not > sure which sounds more natural as English The problem is that, because "checking" is a noun in this sentence, it can't be followed by a direct object so you need "of" to connect "checking" with the thing that is being checked. However, what I would do is rearrange this sentence slightly as to use "checking" as a verb, like this: convert charset table to charmap hash, checking for duplicate source codes along the way While I don't think Michael's suggestion is wrong, I find the above a little more natural. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
Commits
-
Use radix tree for character encoding conversions.
- aeed17d00037 10.0 landed
-
Small fixes to the Perl scripts to create unicode conversion tables.
- bc1686f3f653 10.0 landed
-
Rewrite the perl scripts to produce our Unicode conversion tables.
- 1de9cc0dcca6 10.0 landed
-
Remove leading zeros, for consistency with other map files.
- 6c303223be34 10.0 landed
-
Remove code points < 0x80 from character conversion tables.
- 2c09c93ce1b8 10.0 landed