On 21/9/22 18:07, Alex Bennée wrote:
This helps us construct strings elsewhere before echoing to the
monitor. It avoids having to jump through hoops like:

   monitor_printf(mon, "%s", s->str);

It will be useful in following patches but for now convert all
existing plain "%s" printfs to use the _puts api.

Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

---
v2
   - s/monitor_printf(mon, "%s"/monitor_puts(mon, /
---
  docs/devel/writing-monitor-commands.rst |  2 +-
  include/monitor/monitor.h               |  1 +
  monitor/monitor-internal.h              |  1 -
  block/monitor/block-hmp-cmds.c          | 10 +++++-----
  hw/misc/mos6522.c                       |  2 +-
  monitor/hmp-cmds.c                      |  8 ++++----
  monitor/hmp.c                           |  2 +-
  target/i386/helper.c                    |  2 +-
  8 files changed, 14 insertions(+), 14 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

Reply via email to