[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld |
Date: |
Tue, 26 Jan 2010 13:37:51 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Luiz Capitulino <address@hidden> writes:
> On Mon, 25 Jan 2010 17:38:38 +0200
> "Michael S. Tsirkin" <address@hidden> wrote:
[...]
>> The issue I see isn't related to unsigned. Apparently we currently
>> accept values such as 'a' as valid strings. Since this is not valid json
>> we probably should reject it just in case we will want to switch to
>> another json library, otherwise clients might come to depend on
>> non-standard behaviour.
>
> This extension is only used internally by QEMU
Let me elaborate: when a QEMP client sends us 'a' over the wire, the
parser rejects that as an error. At least that's what we've been
promised when the extension was discussed.
> and we find it
> very convenient otherwise we would have to escape strings in
> dicts and lists, which is error prone and time consuming.
I doubt it would be error prone, but it would sure be annoying and hard
to read. The readability argument is what convinced me.
- [Qemu-devel] [PATCH] win32: use PRId64 instead of %lld, Herve Poussineau, 2010/01/24
- [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/25
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Luiz Capitulino, 2010/01/25
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Markus Armbruster, 2010/01/25
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Luiz Capitulino, 2010/01/25
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/25
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Luiz Capitulino, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld,
Markus Armbruster <=
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Anthony Liguori, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Anthony Liguori, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Anthony Liguori, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Anthony Liguori, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Michael S. Tsirkin, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Anthony Liguori, 2010/01/26
- Re: [Qemu-devel] Re: [PATCH] win32: use PRId64 instead of %lld, Daniel P. Berrange, 2010/01/26