Re: improvements in Unicode tables generation code

From: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
To: hlinnaka(at)iki(dot)fi
Cc: peter(dot)eisentraut(at)enterprisedb(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: improvements in Unicode tables generation code
Date: 2021-06-22 08:33:07
Message-ID: 20210622.173307.2028574862806393946.horikyota.ntt@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

At Tue, 22 Jun 2021 11:20:46 +0300, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> wrote in
> On 22/06/2021 10:20, Peter Eisentraut wrote:
> > v1-0004-Simplify-code-generation-code.patch
> > This simplifies the code a bit, which helps with the next patch.
>
> If we do that, let's add the trailing commas to the other arrays too,
> not just the combined maps.
>
> No objection, but how does this help the next patch?
>
> If we want to avoid the stray commas (and I think they are a little
> ugly, but that's a matter of taste), we could adopt the approach that
> print_radix_table() uses to avoid the comma. That seems simpler than
> what print_from_utf8_combined_map and print_to_utf8_combined_map are
> doing.

+1 for adopting the same method with print_radix_table *if* we do want
to avoid the stray commans.

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2021-06-22 08:53:59 Cosmic ray hits integerset
Previous Message Heikki Linnakangas 2021-06-22 08:20:46 Re: improvements in Unicode tables generation code