[Libvirt-cim] [PATCH] [TEST] Fix none value for disk assigned for Xen
Richard Maciel
rmaciel at linux.vnet.ibm.com
Wed Jan 14 12:20:52 UTC 2009
Guo Lian Yun wrote:
>
> libvirt-cim-bounces at redhat.com wrote on 2009-01-14 04:35:35:
>
> > >> + + if virt == 'LXC':
> > >> d = class_dasd(disk_dev, disk_source, dom_name)
> > >> - + else:
> > >> + # Xen's case falls in here using the default values
> > >> + d = class_dasd(disk_dev, disk_source, dom_name, emu_type)
> > > +1 , but make sure next time we align the comments with the block
> properly.
> > >> class_masd = get_masd_class(virt)
> > >> m = class_masd(
> > >> megabytes=mem_mb,
> > >>
> > > The above function call should be aligned as follows.
> > > m = class_masd(megabytes=mem_mb,
> > > mallocunits=malloc_units,
> > > name=dom_name)
> >
> > I agree with these comments. Daisy - can you resend the patch with
> > these updates?
>
> I think it's better to align them as follows:
> m = class_masd(megabytes=mem_mb,
> mallocunits=malloc_units,
> name=dom_name)
>
> Also, the other functions such as network and processor call have to
> be updated.
> I resend a new patch to fix them.
>
> Thanks!
> >
> > Thanks!
> > --
> > Kaitlin Rupert
> > IBM Linux Technology Center
> > kaitlin at linux.vnet.ibm.com
> >
> > _______________________________________________
> > Libvirt-cim mailing list
> > Libvirt-cim at redhat.com
> > https://www.redhat.com/mailman/listinfo/libvirt-cim
> ------------------------------------------------------------------------
>
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim
I usually follow this style
m = class_masd(megabytes=mem_mb,
mallocunits=malloc_units,
name=dom_name)
(i.e. aligning the function parameters, like in C). But, to be honest,
there isn't a written convention on that and I didn't find any existing
case on the code, so we can follow the style.
--
Richard Maciel, MSc
IBM Linux Technology Center
rmaciel at linux.vnet.ibm.com
More information about the Libvirt-cim
mailing list