<p dir="ltr">I noticed the whitespace changes after I sent this one, and resent it without them shortly after.  Sent a reply that I was going to resend, but it seems not all my emails get through to the list?</p>
<p dir="ltr">How would you recommend I split the changes?  They're all related directly to providing suspend/resume support.</p>
<div class="gmail_quote">On Oct 3, 2014 12:25 AM, "Dietmar Maurer" <<a href="mailto:dietmar@proxmox.com">dietmar@proxmox.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">First, thanks for the patch. But please can you split the patch into smaller ones?<br>
<br>
> Signed-off-by: Dan Hunsaker <<a href="mailto:danhunsaker@gmail.com">danhunsaker@gmail.com</a>><br>
> ---<br>
>  PVE/API2/OpenVZ.pm            | 308 +++++++++++++++++++++++++++---------------<br>
>  PVE/OpenVZ.pm                 |  92 ++++++++-----<br>
>  bin/pvectl                    |  16 ++-<br>
>  www/manager/Utils.js          |  80 +++++------<br>
>  www/manager/openvz/CmdMenu.js |  28 +++-<br>
>  www/manager/qemu/CmdMenu.js   |  26 +++-<br>
>  www/mobile/OpenVzSummary.js   |  30 ++--<br>
>  www/mobile/QemuSummary.js     |  34 +++--<br>
>  8 files changed, 401 insertions(+), 213 deletions(-)<br>
><br>
> diff --git a/PVE/API2/OpenVZ.pm b/PVE/API2/OpenVZ.pm<br>
> index 184ebdf..5d8c0c6 100644<br>
> --- a/PVE/API2/OpenVZ.pm<br>
> +++ b/PVE/API2/OpenVZ.pm<br>
> @@ -71,7 +71,7 @@ my $get_container_storage = sub {<br>
><br>
>  my $check_ct_modify_config_perm = sub {<br>
>      my ($rpcenv, $authuser, $vmid, $pool, $key_list) = @_;<br>
> -<br>
> +<br>
<br>
And there are tons of white-space changes. Please remove them first.<br>
<br>
- Dietmar<br>
<br>
</blockquote></div>