Skip to content

Remove some monitor_printf's

Dr. David Alan Gilbert (git) qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2save_vmstate: Convert to Error** from Monitor *include/sysemu/sysemu.h migration/savevm.c replay/replay-snapshot.c+23 -161 reply
2/2wavcapture: Convert to error_reportaudio/wavcapture.c+17 -222 replies

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchJuan QuintelaGerd HoffmannMarkus Armbruster
SeriesR-b
1/2R-b
2/2R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH 0/2 for 2.10] Remove some monitor_printf's

From: "Dr. David Alan Gilbert" <[email protected]>

I found myself trying to understand cur_mon and found a bunch of uses that could easily be replaced by error_report or Error **.

Dave

Dr. David Alan Gilbert (2):
save_vmstate: Convert to Error** from Monitor *
wavcapture: Convert to error_report

 audio/wavcapture.c       | 39 +++++++++++++++++----------------------
 include/sysemu/sysemu.h  |  2 +-
 migration/savevm.c       | 31 ++++++++++++++++++-------------
 replay/replay-snapshot.c |  6 ++++--
 4 files changed, 40 insertions(+), 38 deletions(-)
Signature · 1 line
2.9.3