From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Jeff Davis <pgsql(at)j-davis(dot)com>, Tatsuo Ishii <ishii(at)postgresql(dot)org>, laurenz(dot)albe(at)wien(dot)gv(dot)at, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: invalidly encoded strings |
Date: | 2007-09-18 12:08:07 |
Message-ID: | 46EFBFA7.7090905@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Tom Lane wrote:
>>
>>> What I think we'd need to have a complete solution is
>>>
>>> convert(text, name) returns bytea
>>> -- convert from DB encoding to arbitrary encoding
>>>
>>> convert(bytea, name, name) returns bytea
>>> -- convert between any two encodings
>>>
>>> convert(bytea, name) returns text
>>> -- convert from arbitrary encoding to DB encoding
>>>
>>> The second and third would need to do a verify step before
>>> converting, of course.
>>>
>> I'm wondering if we should give them disambiguating names, rather than
>> call them all convert.
>>
>
> No. We have a function overloading system, we should use it.
>
>
>
In general I agree with you.
What's bothering me here though is that in the two argument forms, if
the first argument is text the second argument is the destination
encoding, but if the first argument is a bytea the second argument is
the source encoding. That strikes me as likely to be quite confusing,
and we might alleviate that with something like:
text convert_from(bytea, name)
bytea convert_to(text, name)
But if I'm the only one bothered by it I won't worry.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Hannu Krosing | 2007-09-18 12:24:58 | Re: invalidly encoded strings |
Previous Message | Heikki Linnakangas | 2007-09-18 11:18:59 | Re: Open issues for HOT patch |
From | Date | Subject | |
---|---|---|---|
Next Message | Hannu Krosing | 2007-09-18 12:24:58 | Re: invalidly encoded strings |
Previous Message | Bernd Helmle | 2007-09-18 11:50:37 | Show login privilege in psql \du command |