Re: \d+ should display the storage options for columns

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