Re: [HACKERS] Extra Vietnamese unaccent rules - Mailing list pgsql-hackers

From Dang Minh Huong
Subject Re: [HACKERS] Extra Vietnamese unaccent rules
Date
Msg-id 191ea4d0-dbd6-3c79-744b-a976d72f1278@gmail.com
Whole thread Raw
In response to Re: [HACKERS] Extra Vietnamese unaccent rules  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Thanks!


On 2017/08/17 11:56, Tom Lane wrote:
> Michael Paquier <michael.paquier@gmail.com> writes:
>> On Thu, Aug 17, 2017 at 6:01 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> I'm not really qualified to review the Python coding
>>> style, but I did fix a typo in a comment.
>> No pythonist here, but a large confusing "if" condition without any
>> comments is better if split up and explained with comments if that can
>> help in clarifying what the code is doing in any language, so thanks
>> for keeping the code intact.
> Certainly agreed on splitting up the logic into multiple statements.
> I just meant that I don't know enough Python to know if there are
> better ways to do these tests.  (It probably doesn't matter, since
> performance of this script is not an issue, and it's not likely to
> undergo a lot of further development either.)
>
>             regards, tom lane




pgsql-hackers by date:

Previous
From: Douglas Doole
Date:
Subject: Re: [HACKERS] [PATCH] Push limit to sort through a subquery
Next
From: Heikki Linnakangas
Date:
Subject: Re: [HACKERS] SCRAM salt length