Alex Bennée <alex.ben...@linaro.org> writes: > 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); > > Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
There's just one caller other than monitor_vprintf(). Even if you add a few more, I'm not sure keeping it in the interface is worthwhile. If we conclude it is, then this patch should come with a conversion of existing uses of monitor_printf(MON, "%s", STR) to monitor_puts(MON, STR). If we conclude it isn't, switch the lone caller to monitor_printf() and make monitor_puts() static in monitor.c.