Change in vdsm[master]: virt: Vm._getUnderlyingSoundDeviceInfo() moved to core.py
fromani at redhat.com
fromani at redhat.com
Fri Feb 19 08:23:12 UTC 2016
Francesco Romani has posted comments on this change.
Change subject: virt: Vm._getUnderlyingSoundDeviceInfo() moved to core.py
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/53616/2/vdsm/virt/vmdevices/core.py
File vdsm/virt/vmdevices/core.py:
Line 252: :type domain: :class:`DomainDescriptor` instance
Line 253: :param sound_devices: current sound devices for update
Line 254: :type sound_devices: dictionary
Line 255: :param device_conf: current device configuration for update
Line 256: :type device_conf: dictionary
I believe it is sufficient if you have this doc only in the base method, to avoid repeating -and keeping it up to date- on each the subclasses.
Line 257:
Line 258: """
Line 259: for x in domain.get_device_elements('sound'):
Line 260: alias = x.getElementsByTagName('alias')[0].getAttribute('name')
--
To view, visit https://gerrit.ovirt.org/53616
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I05cb24e0a9776e21bd2546d62a56554d74559c61
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzamazal at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list