|
From: | Markus Armbruster |
Subject: | Re: [Qemu-devel] [PATCH 18/20] monitor: Turn int command_mode into bool in_command_mode |
Date: | Tue, 26 May 2015 11:49:39 +0200 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Eric Blake <address@hidden> writes: > On 05/22/2015 05:36 AM, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster <address@hidden> >> --- >> monitor.c | 23 ++++++++++++----------- >> 1 file changed, 12 insertions(+), 11 deletions(-) >> > > Might be worth mentioning in the commit message that... > >> >> -static inline int qmp_cmd_mode(const Monitor *mon) >> -{ >> - return mon->qmp.command_mode; >> -} > > ...qmp_cmd_mode() is now inlined into its callers. Can do. > Nice, fixes up a mis-use of 'int' that I complained about when writing > commit 2d5a8346 > > Reviewed-by: Eric Blake <address@hidden> Thanks!
[Prev in Thread] | Current Thread | [Next in Thread] |