monitor: rename *_pop_one to *_pop_any
authorPeter Xu <peterx@redhat.com>
Wed, 20 Jun 2018 07:32:18 +0000 (15:32 +0800)
committerMarkus Armbruster <armbru@redhat.com>
Sat, 30 Jun 2018 15:50:48 +0000 (17:50 +0200)
commit40687eb741a974c47326ef3cf7f6b25cc0680552
tree87fa7cb798408ea6dcb67bc7fb13c218410739fd
parentd88610111b40bca19925ece0fa81710d425725a8
monitor: rename *_pop_one to *_pop_any

The old names are confusing since both of the old functions are popping
an item from multiple queues rather than a single queue.  In that
sense, *_pop_any() suites better than *_pop_one().

Since at it, touch up the function monitor_qmp_response_pop_any() a bit
to let the callers pass in a QMPResponse struct instead of returning a
struct.  Change the return value to boolean to mark whether we have
popped a valid response instead.

Suggested-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
Message-Id: <20180620073223.31964-3-peterx@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
monitor.c