<div dir="ltr">Thank you :)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 20, 2016 at 6:20 PM, Cole Robinson <span dir="ltr"><<a href="mailto:crobinso@redhat.com" target="_blank">crobinso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 05/20/2016 03:09 AM, Nishith Shah wrote:<br>
> This series moves the parsing of qemu -m memory into a separate function<br>
> and adds suffix support for the -m option.<br>
><br>
> The first patch introduces the function qemuParseCommandLineMem, and the<br>
> second patch handles suffixes for the -m values and adds a test for the<br>
> same.<br>
><br>
> Nishith Shah (2):<br>
>   qemu: parse: Use qemuParseCommandLineMem for -m memory<br>
>   qemu: parse: Handle suffixes for -m memory<br>
><br>
>  src/qemu/qemu_parse_command.c                      | 35 ++++++++++++++++----<br>
>  tests/qemuargv2xmldata/qemuargv2xml-mem-scale.args | 22 +++++++++++++<br>
>  tests/qemuargv2xmldata/qemuargv2xml-mem-scale.xml  | 37 ++++++++++++++++++++++<br>
>  tests/qemuargv2xmltest.c                           |  1 +<br>
>  4 files changed, 88 insertions(+), 7 deletions(-)<br>
>  create mode 100644 tests/qemuargv2xmldata/qemuargv2xml-mem-scale.args<br>
>  create mode 100644 tests/qemuargv2xmldata/qemuargv2xml-mem-scale.xml<br>
><br>
<br>
</span>ACK and pushed now, thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
- Cole<br>
</font></span></blockquote></div><br></div>