[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/14] qapi: Factor out JSON number formattin
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/14] qapi: Factor out JSON number formatting |
Date: |
Wed, 23 Dec 2015 15:32:01 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, 12/21 17:30, Eric Blake wrote:
> Pull out a new qstring_append_json_number() helper, so that all
> JSON output producers can use a consistent style for printing
> floating point without duplicating code (since we are doing more
> data massaging than a simple printf format can handle).
>
> Address one FIXME by adding an Error parameter and warning the
> caller if the requested number cannot be represented in JSON;
> but add another FIXME in its place because we have no way to
> report the problem higher up the stack.
>
> Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
- [Qemu-devel] [PATCH v2 00/14] Add qapi-to-JSON output visitor, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 02/14] qapi: Improve use of qmp/types.h, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 05/14] qapi: Use qstring_append_chr() where appropriate, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 04/14] qapi: Factor out JSON number formatting, Eric Blake, 2015/12/21
- Re: [Qemu-devel] [PATCH v2 04/14] qapi: Factor out JSON number formatting,
Fam Zheng <=
- [Qemu-devel] [PATCH v2 06/14] qapi: Add qstring_append_format(), Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 03/14] qapi: Factor out JSON string escaping, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 08/14] qjson: Simplify by using json-output-visitor, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 10/14] vmstate: use new JSON output visitor, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 07/14] qapi: add json output visitor, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 11/14] qjson: Remove unused file, Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 12/14] qapi: Add qobject_to_json_pretty_prefix(), Eric Blake, 2015/12/21
- [Qemu-devel] [PATCH v2 01/14] qapi: Rename (one) qjson.h to qobject-json.h, Eric Blake, 2015/12/21