Re: \d+ should display the storage options for columns - Mailing list pgsql-patches

From Guillaume Lelarge
Subject Re: \d+ should display the storage options for columns
Date
Msg-id 487C698D.1060200@lelarge.info
Whole thread Raw
In response to Re: \d+ should display the storage options for columns  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
Bruce Momjian a écrit :
> Alvaro Herrera wrote:
>> Bruce Momjian wrote:
>>
>>> Update patch applied;  I also adjusted some translation function calls.
>>> The new output of psql \d+ is:
>>>
>>>     test=> \d+ test
>>>                      Table "public.test"
>>>      Column |  Type   | Modifiers | Storage | Description
>>>     --------+---------+-----------+---------+-------------
>>>      x      | integer |           | plain   |
>>>     Has OIDs: no
>> Thanks for fixing the header problem.
>>
>> This patch introduces other strings, "plain", "main" and so on that are
>> not gettext_noop()ed.  Should we mark those for translation too?  I
>> admit I am not sure, if only because the untranslated strings are what
>> gets passed to ALTER TABLE ... SET STORAGE.  But if that's the
>> decision, then it oughtta be commented in the code ...
>
> I thought about that, but those strings are literal used in our syntax,
> so translating them seemed wrong.
>

I agree on this. If I had to translate these strings, I would copy the
english string in the french translation. I don't see an advantage on
translating these strings.


--
Guillaume.
  http://www.postgresqlfr.org
  http://dalibo.com

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: \d+ should display the storage options for columns
Next
From: Tatsuo Ishii
Date:
Subject: Re: [HACKERS] WITH RECURSIVE updated to CVS TIP