Re: [PATCH 2/7] qemu: rename some functions in qemu_command.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/02/2013 07:01 PM, Eric Blake wrote:
> On 08/02/2013 10:55 AM, Laine Stump wrote:
>> * qemuDomainPCIAddressSetNextAddr
>>
>> The name of this function was confusing because 1) other functions in
>> the file that end in "Addr" are only operating on a single function of
>> one PCI slot, not the entire slot, while functions that do something
>> with the entire slot end in "Slot", and 2) it didn't contain a verb
>> describing what it is doing (the "Set" refers to the set that contains
>> all PCI buses in the system, used to keep track of which slots in
>> which buses are already reserved for use).
>>
>> It is now renamed to qemuDomainPCIAddressReserveNextSlot, which more
>> clearly describes what it is doing. Arguably, it could have been
>> changed to qemuDomainPCIAddressSetReserveNextSlot, but 1) the word
>> "set" is confusing in this context because it could be intended as a
>> verb or as a noun, and 2) most other functions that operate on a
>> single slot or address within this set are also named
>> qemuDomainPCIAddress... rather than qemuDomainPCIAddressSet... Only
>> the Create, Free, and Grow functions for an address set (which modify the
>> entire set, not just one element) use "Set" in their name.
>>
>> * qemuPCIAddressAsString, qemuPCIAddressValidate
>>
>> All the other functions in this set are named
>> qemuDomainPCIAddressxxxxx, so I renamed these to be consistent.
>> ---
>>  src/qemu/qemu_command.c | 78 ++++++++++++++++++++++++++++++-------------------
>>  src/qemu/qemu_command.h |  6 ++--
>>  2 files changed, 51 insertions(+), 33 deletions(-)
> Not a 1:1 line mapping; but I checked that it's all because of line
> re-wrapping now that the names are longer.
>
> ACK.
>


Thanks. This one is pushed too (and also eliminates the need for one of
Cole's ARM support patch series)

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]