Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-21 Thread Marc-André Lureau
Hi On Tue, Mar 21, 2017 at 9:06 PM Eric Blake wrote: > On 03/21/2017 11:49 AM, Markus Armbruster wrote: > > > > > QMP clients that work around the "large positive integers are rejected" > > bugs by sending large negative ones instead may well exist. Fixing the > > interface

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-21 Thread Eric Blake
On 03/21/2017 11:49 AM, Markus Armbruster wrote: > > QMP clients that work around the "large positive integers are rejected" > bugs by sending large negative ones instead may well exist. Fixing the > interface would break them. Depressing. Eric, could you have a peek at > libvirt? Yes,

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-21 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Mon, Mar 13, 2017 at 5:29 PM Marc-André Lureau > wrote: > > Hi > > - Original Message - >> On 03/13/2017 02:15 AM, Markus Armbruster wrote: >> > Eric Blake writes: >> > >> >> On

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-21 Thread Marc-André Lureau
Hi On Mon, Mar 13, 2017 at 5:29 PM Marc-André Lureau wrote: Hi - Original Message - > On 03/13/2017 02:15 AM, Markus Armbruster wrote: > > Eric Blake writes: > > > >> On 03/11/2017 07:22 AM, Marc-André Lureau wrote: > >>> The type is not used at

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-13 Thread Marc-André Lureau
Hi - Original Message - > On 03/13/2017 02:15 AM, Markus Armbruster wrote: > > Eric Blake writes: > > > >> On 03/11/2017 07:22 AM, Marc-André Lureau wrote: > >>> The type is not used at all yet. Add some tests to exercice it. > >> > >> s/exercice/exercise/ > >> > >> I

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-13 Thread Eric Blake
On 03/13/2017 02:15 AM, Markus Armbruster wrote: > Eric Blake writes: > >> On 03/11/2017 07:22 AM, Marc-André Lureau wrote: >>> The type is not used at all yet. Add some tests to exercice it. >> >> s/exercice/exercise/ >> >> I wonder if we need this patch at all. >> >> I've

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-13 Thread Markus Armbruster
Eric Blake writes: > On 03/11/2017 07:22 AM, Marc-André Lureau wrote: >> The type is not used at all yet. Add some tests to exercice it. > > s/exercice/exercise/ > > I wonder if we need this patch at all. > > I've been thinking about a possible alternative representation, such

Re: [Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-11 Thread Eric Blake
On 03/11/2017 07:22 AM, Marc-André Lureau wrote: > The type is not used at all yet. Add some tests to exercice it. s/exercice/exercise/ I wonder if we need this patch at all. I've been thinking about a possible alternative representation, such that a single QInt type can cover _both_ signed and

[Qemu-devel] [PATCH 04/21] qobject: add quint type

2017-03-11 Thread Marc-André Lureau
The type is not used at all yet. Add some tests to exercice it. Signed-off-by: Marc-André Lureau --- qapi/introspect.json | 2 +- scripts/qapi.py | 28 --- include/qapi/qmp/quint.h | 25