[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor |
Date: |
Wed, 18 May 2016 09:24:02 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 05/18/2016 09:16 AM, Eric Blake wrote:
>>> +static void json_output_type_any(Visitor *v, const char *name, QObject
>>> **obj,
>>> + Error **errp)
>>> +{
>>> + JsonOutputVisitor *jov = to_jov(v);
>>> + QString *str = qobject_to_json(*obj);
>>> + assert(str);
>>
>> Can't happen.
>
> Can too. From tests/check-qobject-json.c:
>
> obj = QOBJECT(qstring_from_str(utf8_in));
> str = qobject_to_json(obj);
> if (json_out) {
> g_assert(str);
> g_assert_cmpstr(qstring_get_str(str), ==, json_out);
> } else {
> g_assert(!str);
> }
>
> where the failures occur when it is impossible to output proper UTF-8
> due to invalid encoding in a string.
Correction - that test has dead code, because:
json_out = test_cases[i].json_out ?: test_cases[i].json_in;
so you are right after all - we currently cannot fail on a conversion to
JSON (although the result might not be valid JSON). At any rate, my
conclusion remains:
> Arguably, that case would be nicer
> if it could set an Error* (and would make my argument for setting an
> error on Inf/NaN for numbers also a bit more tenable), but that is
> additional work that I haven't tackled yet. I'm trying to get the series
> posted for another round of review, where I've done some major
> reshuffling (such as doing the clone visitor first, not second, in the
> series), so hopefully later today.
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Markus Armbruster, 2016/05/02
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Eric Blake, 2016/05/02
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Markus Armbruster, 2016/05/03
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Markus Armbruster, 2016/05/04
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Eric Blake, 2016/05/06
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Markus Armbruster, 2016/05/06
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Eric Blake, 2016/05/06
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Eric Blake, 2016/05/10
Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor, Eric Blake, 2016/05/18
- Re: [Qemu-devel] [PATCH v3 07/18] qapi: Add json output visitor,
Eric Blake <=