[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/12] qapi: add human mode to StringOutputVisit
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 07/12] qapi: add human mode to StringOutputVisitor |
Date: |
Thu, 30 Jan 2014 15:12:32 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Il 30/01/2014 15:09, Eric Blake ha scritto:
| Also, I like how your int printout was both decimal and hex; but
| here you are throwing away information (and the bigger the number,
| the more we lose as to how much got rounded away). I'd rather see
| this as:
|
| "%0.03f%c (%llu)"
|
| so that we also have access to the unrounded exact amount.
Perhaps the other way round (since hex is in parentheses)?
This patch is just moving the code from qdev-properties.c. I agree
with all your suggestion, but I'd prefer to tackle it as follow-ups or
as patch 13/12. There are other problems, like 512MB printed as
0.500G (which is correct but looks weird).
Paolo
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJS6l3QAAoJEBvWZb6bTYbyiSAQAIUXn6u0GTdSI6KbWxF1hOpC
nXno3OekuKomBMPZxUYzXKztwz3LhKKrPpuFHPpcIXvdIENZ7tmzTAoJFaUWCvIl
R4wCPpNRiSAI5wrF8CnqPzirsc+9SlKrFCqP+mjwYQ56vGhmKcvzT0uAlLdHMOyB
dn1g3PH2z8RNKSf1WxCViRwgHrSBa+w/ZjWrVQ8yEJHfBPTlQNA/x+cKKBcIQA5v
SmYGUSCWj84cgEs9o9psLefxkSHyzWN+OoH1zU8VL5qegwFeQYGbHC8Q2zlM6hZO
G1EtSwEknhPPTOP7+muRgNZaVaZtBi4irkZ677OtdXgrudeGaSlHnVsuYJQ72+tv
B2A6DuFq4kdLB3eV/+FPnpZk1ky896nDguFuj2A8Yo7SJr88JszcI/OAosW0mF6d
cML2NEt2AbwMUccbpr43qmkiLDKN0G1MIN4BVAukSGCdQfU5X6DRWWZIuicKzBbf
VPYjK7puI57wC8L2Xjj8fTW6PbBG09R6IJuDvnYLRubY48qTQupOLlxx16sw61EV
450s+QkZmRTPP3GmwfsNiz+/rDlpqNB/iWnVFrek3K9Lsz0eM/CNYqiyG+LUVBCB
e9D2ktLZ2mmKahiwQY4lUP4UYFjHLwhYly+OefznhYq/gKthnQD3jNaHtf3ZkNaW
HO4JZAjvTsFddSr7cCTD
=LRTe
-----END PGP SIGNATURE-----
- [Qemu-devel] [PATCH 03/12] qdev: remove legacy parsers for hex8/32/64, (continued)
- [Qemu-devel] [PATCH 03/12] qdev: remove legacy parsers for hex8/32/64, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 04/12] qdev: legacy properties are now read-only, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 05/12] qdev: legacy properties are just strings, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 06/12] qdev: inline qdev_prop_parse, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 07/12] qapi: add human mode to StringOutputVisitor, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 08/12] qdev: use human mode in "info qtree", Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 09/12] qdev: remove most legacy printers, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 10/12] qdev: remove hex8/32/64 property types, Paolo Bonzini, 2014/01/30
- [Qemu-devel] [PATCH 11/12] qdev: add enum property types to QAPI schema, Paolo Bonzini, 2014/01/30