On Wed, 2015-07-22 at 11:13 +0100, Wei Liu wrote:
> 
> +# Mode should be either 1 ("local") or 0 ("remote")
> +our ($whhost, $gn, $mode) = @ARGV;

I think $mode would be better given a more "boolean" name, in this case
$local.

However that's a minor nit so with or without that changed:

Acked-by: Ian Campbell <ian.campb...@citrix.com>

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to