Re: badly calculated width of emoji in psql

Поиск
Список
Период
Сортировка
От John Naylor
Тема Re: badly calculated width of emoji in psql
Дата
Msg-id CAFBsxsFQ=scF=hgvrUWwDBzf1ZSkdmq1XRN4NJPMtniqNfpBCQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: badly calculated width of emoji in psql  (John Naylor <john.naylor@enterprisedb.com>)
Ответы Re: badly calculated width of emoji in psql
Re: badly calculated width of emoji in psql
Список pgsql-hackers
The patch looks pretty good to me. I just have a stylistic suggestion which I've attached as a text file. There are also some outdated comments that are not the responsibility of this patch, but I kind of want to fix them now:

 *  - Hangul Jamo medial vowels and final consonants (U+1160-U+11FF)
 * have a column width of 0.

We got rid of this range in d8594d123c1, which is correct.

 *  - Other format characters (general category code Cf in the Unicode
 * database) and ZERO WIDTH SPACE (U+200B) have a column width of 0.

We don't treat Cf the same as Me or Mn, and I believe that's deliberate. We also no longer have the exception for zero-width space.

It seems the consensus so far is that performance is not an issue, and I'm inclined to agree.

I'm a bit concerned about the build dependencies not working right, but it's not clear it's even due to the patch. I'll spend some time investigating next week.

--
Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Default to TIMESTAMP WITH TIME ZONE?
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: Worth using personality(ADDR_NO_RANDOMIZE) for EXEC_BACKEND on linux?