2010/9/1 Eric Blake <[email protected]>: > * src/vbox/vbox_tmpl.c (vboxDomainDefineXML): Split... > (vboxSetBootDeviceOrder, vboxAttachDrives, vboxAttachSound) > (vboxAttachNetwork, vboxAttachSerial, vboxAttachParallel) > (vboxAttachVideo, vboxAttachDisplay, vboxAttachUSB): ...into new > helper functions. > --- > > Should be no functional changes - just breaking up a HUGE function > into more manageable chunks with less indentation. > > In reviewing this, it may help to apply the patch and use > 'git diff -b'. >
That helps a lot and it reveals that a major part of the remaining diff is just breaking long lines. You could have split that into a separate patch, but it's okay as it is. ACK. Mathias -- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
