Re: [Qemu-devel] [PATCH 08/17] qapi: update the qobject visitor to use QUInt

2017-05-31 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Tue, May 16, 2017 at 9:33 PM Markus Armbruster wrote: > >> On the subject: there is no such thing as "QUInt". I guess you mean >> "uint type" (like in PATCH 06's subject). Could also say "QNUM_U64". >> >>

Re: [Qemu-devel] [PATCH 08/17] qapi: update the qobject visitor to use QUInt

2017-05-30 Thread Marc-André Lureau
Hi On Tue, May 16, 2017 at 9:33 PM Markus Armbruster wrote: > On the subject: there is no such thing as "QUInt". I guess you mean > "uint type" (like in PATCH 06's subject). Could also say "QNUM_U64". > > Apropos subject: humor me, and start your subjects with a capital >

Re: [Qemu-devel] [PATCH 08/17] qapi: update the qobject visitor to use QUInt

2017-05-17 Thread Markus Armbruster
Markus Armbruster writes: > On the subject: there is no such thing as "QUInt". I guess you mean > "uint type" (like in PATCH 06's subject). Could also say "QNUM_U64". > > Apropos subject: humor me, and start your subjects with a capital > letter, like this: > > qapi:

Re: [Qemu-devel] [PATCH 08/17] qapi: update the qobject visitor to use QUInt

2017-05-16 Thread Markus Armbruster
On the subject: there is no such thing as "QUInt". I guess you mean "uint type" (like in PATCH 06's subject). Could also say "QNUM_U64". Apropos subject: humor me, and start your subjects with a capital letter, like this: qapi: Update the qobject visitor ... Marc-André Lureau

[Qemu-devel] [PATCH 08/17] qapi: update the qobject visitor to use QUInt

2017-05-09 Thread Marc-André Lureau
Switch to use QNum/uint where appropriate to remove i64 limitation. The input visitor will cast i64 input to u64 for compatibility reasons (existing json QMP client already use negative i64 for large u64, and expect an implicit cast in qemu). Signed-off-by: Marc-André Lureau