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
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 |