<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2009-01-14
04:35:35:<br>
<br>
> >> +     +    if virt == 'LXC':<br>
> >>          d = class_dasd(disk_dev,
disk_source, dom_name)<br>
> >> -    +    else:<br>
> >> +    # Xen's case falls in here using the default
values<br>
> >> +        d = class_dasd(disk_dev, disk_source,
dom_name, emu_type)   <br>
> > +1 , but make sure next time we align the comments with the block
properly.<br>
> >>      class_masd = get_masd_class(virt)<br>
> >>      m = class_masd(<br>
> >>                  megabytes=mem_mb,<br>
> >>   <br>
> > The above function call should be aligned as follows.<br>
> > m = class_masd(megabytes=mem_mb,<br>
> > mallocunits=malloc_units,<br>
> > name=dom_name)<br>
> <br>
> I agree with these comments.  Daisy - can you resend the patch
with <br>
> these updates?</font></tt>
<br>
<br><tt><font size=2>  I think it's better to align them as follows:</font></tt>
<br><tt><font size=2>  m = class_masd(megabytes=mem_mb,<br>
                 mallocunits=malloc_units,<br>
                 name=dom_name)
  </font></tt>
<br>
<br><tt><font size=2>  Also, the other functions such as network and
processor call have to be updated.</font></tt>
<br><tt><font size=2>  I resend a new patch to fix them.</font></tt>
<br>
<br><tt><font size=2>  Thanks!<br>
> <br>
> Thanks!<br>
> -- <br>
> Kaitlin Rupert<br>
> IBM Linux Technology Center<br>
> kaitlin@linux.vnet.ibm.com<br>
> <br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>