Re: broken table formatting in psql

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: John Naylor <john(dot)naylor(at)enterprisedb(dot)com>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: broken table formatting in psql
Date: 2022-09-08 05:50:45
Message-ID: CAFj8pRBnc1CzSBF3JiMOC9UTbTZW8nDw4m+=dsWKzRVF-_brJQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

čt 8. 9. 2022 v 7:39 odesílatel John Naylor <john(dot)naylor(at)enterprisedb(dot)com>
napsal:

> On Fri, Sep 2, 2022 at 3:19 PM Kyotaro Horiguchi
> <horikyota(dot)ntt(at)gmail(dot)com> wrote:
> >
> > At Fri, 2 Sep 2022 13:43:50 +0700, John Naylor <
> john(dot)naylor(at)enterprisedb(dot)com> wrote in
> > > If there is any doubt about including all of Cf, we could also just
> > > add a branch in wchar.c to hard-code the 200B-200F range.
> >
> > If every way has defect to the similar extent, I think we will choose
> > to use authoritative data at least for the first step. We might want
> > to have additional filtering on it but it would be another issue,
> > maybe.
> >
> > Attached is the first cut of that. (The commit messages is not great,
> > though.)
>
> Okay, the patch looks good to me overall. Comparing releases, some
> other ranges were in v11 but left out in v12 with the transition to
> using a script:
>
> 0x070F
> {0x200B, 0x200F}
> {0x202A, 0x202E}
> {0x206A, 0x206F}
> 0xFEFF
> {0xFFF9, 0xFFFB}
>
> Does anyone want to advocate for backpatching these missing ranges to
> v12 and up? v12 still has a table in-line so trivial to remedy, but
> v13 and up use a script, so these exceptions would likely have to use
> hard-coded branches to keep from bringing in new changes.
>
> If so, does anyone want to advocate for including this patch in v15?
> It claims Unicode 14.0.0, and this would make that claim more
> technically correct as well as avoiding additional branches.
>

I think it can be fixed just in v15 and master. This issue has no impact
on SQL.

Thank you for fixing this issue

Regards

Pavel

> --
> John Naylor
> EDB: http://www.enterprisedb.com
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Drouvot, Bertrand 2022-09-08 06:49:13 Re: [PATCH] Query Jumbling for CALL and SET utility statements
Previous Message Nathan Bossart 2022-09-08 05:50:35 Re: predefined role(s) for VACUUM and ANALYZE