diff options
| author | Markus Armbruster <armbru@redhat.com> | 2019-04-17 21:17:56 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2019-04-18 22:18:59 +0200 |
| commit | b6b71cb5c674a97a4cd935349ce8a2764f720af4 (patch) | |
| tree | e19df340f498ea7caa5bf81c756c02109042258b /monitor.c | |
| parent | e1ce7d747bc071f86a77683b433b852001163c2c (diff) | |
| download | focaccia-qemu-b6b71cb5c674a97a4cd935349ce8a2764f720af4.tar.gz focaccia-qemu-b6b71cb5c674a97a4cd935349ce8a2764f720af4.zip | |
memory: Clean up how mtree_info() prints
mtree_info() takes an fprintf()-like callback and a FILE * to pass to it, and so do its helper functions. Passing around callback and argument is rather tiresome. Its only caller hmp_info_mtree() passes monitor_printf() cast to fprintf_function and the current monitor cast to FILE *. The type-punning is technically undefined behaviour, but works in practice. Clean up: drop the callback, and call qemu_printf() instead. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Message-Id: <20190417191805.28198-9-armbru@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | monitor.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/monitor.c b/monitor.c index 1650ceec3a..0819b99ef7 100644 --- a/monitor.c +++ b/monitor.c @@ -1907,8 +1907,7 @@ static void hmp_info_mtree(Monitor *mon, const QDict *qdict) bool dispatch_tree = qdict_get_try_bool(qdict, "dispatch_tree", false); bool owner = qdict_get_try_bool(qdict, "owner", false); - mtree_info((fprintf_function)monitor_printf, mon, flatview, dispatch_tree, - owner); + mtree_info(flatview, dispatch_tree, owner); } static void hmp_info_numa(Monitor *mon, const QDict *qdict) |