Re: GUC names in messages

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: GUC names in messages
Дата
Msg-id 202312081255.wlsfmhe2sri7@alvherre.pgsql
обсуждение исходный текст
Ответ на Re: GUC names in messages  (Peter Smith <smithpb2250@gmail.com>)
Ответы Re: GUC names in messages  (Peter Smith <smithpb2250@gmail.com>)
Список pgsql-hackers
On 2023-Dec-08, Peter Smith wrote:

> Patch 0001 -- "datestyle" becomes DateStyle in messages
> Rebased this again, which was part of an earlier patch set
> - I think any GUC names documented as MixedCase should keep that same
> case in messages; this also obeys the guidelines recently pushed [1].

I agree.

> Patch 0002 -- use mixed case for intervalstyle error message
> I found that the GUC name substituted to the error message was coming
> from the statement, not from the original name in the guc_tables, so
> there was a case mismatch:

I agree.  Let's also add a test that shows this difference (my 0002
here).

I'm annoyed that this saga has transiently created a few untranslated
strings by removing unnecessary quotes but failing to move the variable
names outside the translatable part of the string.  I change a few of
those in 0003 -- mostly the ones in strings already touched by commit
8d9978a7176a, but also a few others.  Didn't go out of my way to grep
for other possible messages to fix, though.  (I feel like this is
missing some "translator:" comments.)

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
"We’ve narrowed the problem down to the customer’s pants being in a situation
 of vigorous combustion" (Robert Haas, Postgres expert extraordinaire)

Вложения

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

Предыдущее
От: Nikolay Shaplov
Дата:
Сообщение: Re: [PATCH] New [relation] option engine
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: [PATCH] New [relation] option engine