δΊ 2014/5/1 22:44, Eric Blake ει:
On 04/30/2014 10:26 PM, Wenchao Xia wrote:
Signed-off-by: Wenchao Xia <wenchaoq...@gmail.com>
---
vl.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
This, and many of the subsequent patches in the series, are a bit
misleading; I have to go back to 5/28 to make sure that both pieces of
the conversion are in sync. Is there a technical reason why you can't
provide both the qapi definition and the conversion to the new
qapi_event_send_FOO function in the same patch, one patch per FOO,
rather than your current split?
No, it is just easy to get conflict to modifiy one file in 20+
patches when rebase, so I modified it once in patch 5/28. I'll adjust
it in next version.
