On 3/6/20 2:19 PM, Tom Lane wrote:
>> Maybe Chapman has a use case in mind he can test with? Barring that,
>> the patch seems ready for commit.
>
> I went ahead and pushed this, just to get it out of my queue.
> Chapman's certainly welcome to kibitz some more of course.
Sorry, yeah, I don't think
John Naylor writes:
> Not this patch's job perhaps, but now that check_unicode_value() only
> depends on the input, maybe it can be put into pgwchar.h with other
> static inline helper functions? That test is duplicated in
> addunicode() and pg_unicode_to_server(). Maybe:
> static inline bool
> c
On Tue, Feb 25, 2020 at 1:49 AM Tom Lane wrote:
>
> I wrote:
> > [ unicode-escapes-with-other-server-encodings-2.patch ]
>
> I see this patch got sideswiped by the recent refactoring of JSON
> lexing. Here's an attempt at fixing it up. Since the frontend
> code isn't going to have access to enco
On Mon, Feb 24, 2020 at 11:19 PM Tom Lane wrote:
> I see this patch got sideswiped by the recent refactoring of JSON
> lexing. Here's an attempt at fixing it up. Since the frontend
> code isn't going to have access to encoding conversion facilities,
> this creates a difference between frontend a
I wrote:
> [ unicode-escapes-with-other-server-encodings-2.patch ]
I see this patch got sideswiped by the recent refactoring of JSON
lexing. Here's an attempt at fixing it up. Since the frontend
code isn't going to have access to encoding conversion facilities,
this creates a difference between
I wrote:
> Andrew Dunstan writes:
>> Perhaps I expressed myself badly. What I meant was that we should keep
>> the json and text escape rules in sync, as they are now. Since we're
>> changing the text rules to allow resolvable non-ascii unicode escapes
>> in non-utf8 locales, we should do the same
Andrew Dunstan writes:
> Perhaps I expressed myself badly. What I meant was that we should keep
> the json and text escape rules in sync, as they are now. Since we're
> changing the text rules to allow resolvable non-ascii unicode escapes
> in non-utf8 locales, we should do the same for json.
Got
On Wed, Jan 15, 2020 at 7:55 AM Tom Lane wrote:
>
> Andrew Dunstan writes:
> > On Wed, Jan 15, 2020 at 4:25 AM Chapman Flack wrote:
> >> On 1/14/20 10:10 AM, Tom Lane wrote:
> >>> to me that this error is just useless pedantry. As long as the DB
> >>> encoding can represent the desired characte
On 1/14/20 4:25 PM, Tom Lane wrote:
> Andrew Dunstan writes:
>> On Wed, Jan 15, 2020 at 4:25 AM Chapman Flack wrote:
>>> On 1/14/20 10:10 AM, Tom Lane wrote:
to me that this error is just useless pedantry. As long as the DB
encoding can represent the desired character, it should be tra
Andrew Dunstan writes:
> On Wed, Jan 15, 2020 at 4:25 AM Chapman Flack wrote:
>> On 1/14/20 10:10 AM, Tom Lane wrote:
>>> to me that this error is just useless pedantry. As long as the DB
>>> encoding can represent the desired character, it should be transparent
>>> to users.
>> That's my posit
On Wed, Jan 15, 2020 at 4:25 AM Chapman Flack wrote:
>
> On 1/14/20 10:10 AM, Tom Lane wrote:
> > to me that this error is just useless pedantry. As long as the DB
> > encoding can represent the desired character, it should be transparent
> > to users.
>
> That's my position too.
>
and mine.
c
On 1/14/20 10:10 AM, Tom Lane wrote:
> to me that this error is just useless pedantry. As long as the DB
> encoding can represent the desired character, it should be transparent
> to users.
That's my position too.
Regards,
-Chap
I wrote:
> Andrew Dunstan writes:
>> On Tue, Jan 14, 2020 at 10:02 AM Tom Lane wrote:
>>> Grepping for other direct uses of unicode_to_utf8(), I notice that
>>> there are a couple of places in the JSON code where we have a similar
>>> restriction that you can only write a Unicode escape in UTF8 s
Andrew Dunstan writes:
> On Tue, Jan 14, 2020 at 10:02 AM Tom Lane wrote:
>> Grepping for other direct uses of unicode_to_utf8(), I notice that
>> there are a couple of places in the JSON code where we have a similar
>> restriction that you can only write a Unicode escape in UTF8 server
>> encodi
On Tue, Jan 14, 2020 at 10:02 AM Tom Lane wrote:
>
>
> Grepping for other direct uses of unicode_to_utf8(), I notice that
> there are a couple of places in the JSON code where we have a similar
> restriction that you can only write a Unicode escape in UTF8 server
> encoding. I'm not sure whether
I threatened to do this in another thread [1], so here it is.
This patch removes the restriction that the server encoding must
be UTF-8 in order to write any Unicode escape with a value outside
the ASCII range. Instead, we'll allow the notation and convert to
the server encoding if that's possibl
16 matches
Mail list logo