
2 Sep
2010
2 Sep
'10
1:25 p.m.
2010/9/1 Eric Blake <eblake@redhat.com>:
* 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