[Libvir] [PATCH] Add error message to virsh setmem/setmaxmem

Daniel Veillard veillard at redhat.com
Tue Mar 20 15:31:55 UTC 2007


On Tue, Mar 20, 2007 at 08:35:08AM +0900, Masayuki Sunou wrote:
> Hi
> 
> Virsh setmem/setmaxmem doesn't output an error message when an invalid
> value is set to "bytes".
> 
> This patch outputs an error message, when an invalid value is set as
> "bytes" of virsh setmem/setmaxmem. 

  Okay, makes sense, but I changed the message to

  vshError(ctl, FALSE, _("Invalid value of %d for memory size"), bytes);

in order to provide more contextual informations,

  thanks !

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard at redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/




More information about the libvir-list mailing list