On 04/16/2018 01:53 PM, John Ferlan wrote:
On 04/13/2018 10:47 AM, Michal Privoznik wrote:
> In next patches this name will be needed for a different memeber.
s/memeber/member
> Also, it makes sense to rename the variable because it does not
> contain reference to parent device, just its name.
>
> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
> ---
> src/conf/virnodedeviceobj.c | 2 +-
> src/datatypes.c | 2 +-
> src/datatypes.h | 2 +-
> src/libvirt-nodedev.c | 6 +++---
> src/node_device/node_device_driver.c | 4 ++--
> src/test/test_driver.c | 6 +++---
> 6 files changed, 11 insertions(+), 11 deletions(-)
>
[...]
FWIW: Not sure about changing the accessor's name too - that would mean
the API name doesn't match (virNodeDeviceGetParent which could lead to
other problems with automatic code generation).
Not sure what do you mean by code generation. But that reminds me I
forgot to rename variable on RPC protocol level :-)
Michal