From: Konstantin Osipov <kostja@tarantool.org>
To: tarantool-patches@freelists.org
Cc: vdavydov.dev@gmail.com, alexander.turenko@tarantool.org,
Serge Petrenko <sergepetrenko@tarantool.org>
Subject: Re: [tarantool-patches] [PATCH] Extend range of printable unicode characters
Date: Tue, 16 Jul 2019 21:31:57 +0300 [thread overview]
Message-ID: <20190716183157.GB19727@atlas> (raw)
In-Reply-To: <20190716134331.15327-1-sergepetrenko@tarantool.org>
* Serge Petrenko <sergepetrenko@tarantool.org> [19/07/16 16:44]:
> Before the patch IS_PRINTABLE macros was used
> to determine if given character is printable or not.
> This macros did not take into account characters
> encoded with 4 bytes.
> After the patch IS_PRINTABLE is replaced with new
> corresponding function. Now the range of printable
> characters is: (libyaml old range) U (icu range). This
> new range include characters encoded with 4 bytes.
>
Please don't forget to upstream the patch to the maintainer once
it's reviewed.
Has there been any changes in the upstream which we should be
aware of?
On another note, since we're switching to Lua emitter as the
defualt one quite soon, we can perhaps close #4090 as won't fix
rather than bother improving libyaml.
> Related to tarantool/tarantool#4090
--
Konstantin Osipov, Moscow, Russia
next prev parent reply other threads:[~2019-07-16 18:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-16 13:43 Serge Petrenko
2019-07-16 18:31 ` Konstantin Osipov [this message]
2019-07-17 9:00 ` [tarantool-patches] " Serge Petrenko
2019-07-18 4:50 ` Kirill Yukhin
2019-07-18 9:49 ` [tarantool-patches] " Konstantin Osipov
2019-07-18 11:16 ` Kirill Yukhin
2019-07-18 11:38 ` Konstantin Osipov
-- strict thread matches above, loose matches on Subject: below --
2019-07-18 8:39 Serge Petrenko
2019-07-19 12:04 ` [tarantool-patches] " Kirill Yukhin
2019-07-02 11:33 Ivan Koptelov
2019-07-02 11:32 Ivan Koptelov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190716183157.GB19727@atlas \
--to=kostja@tarantool.org \
--cc=alexander.turenko@tarantool.org \
--cc=sergepetrenko@tarantool.org \
--cc=tarantool-patches@freelists.org \
--cc=vdavydov.dev@gmail.com \
--subject='Re: [tarantool-patches] [PATCH] Extend range of printable unicode characters' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox