On 10/16/2017 04:20 PM, Kothapally Madhu Pavan wrote: > Move qemuFreeKeywords into qemu_parse_command.c as > qemuKeywordsFree and call it rather than inline code > in multiple places. To be consistent, the new API would be qemuParseKeywordsFree - API's in qemu_parse_command.c start with qemuParse I've fixed that up as well as this description and pushed. Thanks for doing this quickly. Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx> (and pushed) John > > Signed-off-by: Kothapally Madhu Pavan <kmp@xxxxxxxxxxxxxxxxxx> > --- > src/qemu/qemu_monitor_json.c | 15 ++----------- > src/qemu/qemu_parse_command.c | 52 ++++++++++++++++--------------------------- > src/qemu/qemu_parse_command.h | 5 +++++ > 3 files changed, 26 insertions(+), 46 deletions(-) > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list