Re: [libvirt] [PATCH 1/1] includes: function parameter names same in headers
On Mon, Feb 12, 2018 at 06:35:45PM +, Daniel P. Berrangé wrote: > On Mon, Feb 12, 2018 at 12:20:55PM -0600, Chris Venteicher wrote: > > Headers use same function parameter names as definition code. > > --- > > include/libvirt/libvirt-domain.h| 26 +- > > include/libvirt/libvirt-event.h | 4 ++-- > > include/libvirt/libvirt-host.h | 4 ++-- > > include/libvirt/libvirt-interface.h | 4 ++-- > > include/libvirt/libvirt-network.h | 6 +++--- > > include/libvirt/libvirt-nwfilter.h | 2 +- > > include/libvirt/libvirt-qemu.h | 2 +- > > include/libvirt/libvirt-secret.h| 4 ++-- > > include/libvirt/libvirt-storage.h | 12 ++-- > > include/libvirt/libvirt-stream.h| 22 +++--- > > 10 files changed, 43 insertions(+), 43 deletions(-) > > > > diff --git a/include/libvirt/libvirt-domain.h > > b/include/libvirt/libvirt-domain.h > > index 4048acf38..60ec35d87 100644 > > --- a/include/libvirt/libvirt-domain.h > > +++ b/include/libvirt/libvirt-domain.h > > > @@ -1161,11 +1161,11 @@ int virDomainFree > > (virDomainPtr domain); > > */ > > int virDomainSuspend(virDomainPtr domain); > > int virDomainResume (virDomainPtr domain); > > -int virDomainPMSuspendForDuration (virDomainPtr domain, > > +int virDomainPMSuspendForDuration (virDomainPtr dom, > > This changes domain -> dom , but then > > > > > -int virDomainSetInterfaceParameters (virDomainPtr dom, > > +int virDomainSetInterfaceParameters (virDomainPtr > > domain, > > This changes dom -> domain. > > IMHO this is no better than we started with. We should aim for consistency > naming, so rather than only changing the .h, we should change .c and .h > to match preferred naming. For that matter, we should add a custom syntx-check rule to cfg.mak to validate the variable naming for various data types. ie validate that virDomainPtr is always "dom", not "domain". virNetworkPtr is "net" not "network", etc Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 1/1] includes: function parameter names same in headers
On Mon, Feb 12, 2018 at 12:20:55PM -0600, Chris Venteicher wrote: > Headers use same function parameter names as definition code. > --- > include/libvirt/libvirt-domain.h| 26 +- > include/libvirt/libvirt-event.h | 4 ++-- > include/libvirt/libvirt-host.h | 4 ++-- > include/libvirt/libvirt-interface.h | 4 ++-- > include/libvirt/libvirt-network.h | 6 +++--- > include/libvirt/libvirt-nwfilter.h | 2 +- > include/libvirt/libvirt-qemu.h | 2 +- > include/libvirt/libvirt-secret.h| 4 ++-- > include/libvirt/libvirt-storage.h | 12 ++-- > include/libvirt/libvirt-stream.h| 22 +++--- > 10 files changed, 43 insertions(+), 43 deletions(-) > > diff --git a/include/libvirt/libvirt-domain.h > b/include/libvirt/libvirt-domain.h > index 4048acf38..60ec35d87 100644 > --- a/include/libvirt/libvirt-domain.h > +++ b/include/libvirt/libvirt-domain.h > @@ -1161,11 +1161,11 @@ int virDomainFree > (virDomainPtr domain); > */ > int virDomainSuspend(virDomainPtr domain); > int virDomainResume (virDomainPtr domain); > -int virDomainPMSuspendForDuration (virDomainPtr domain, > +int virDomainPMSuspendForDuration (virDomainPtr dom, This changes domain -> dom , but then > > -int virDomainSetInterfaceParameters (virDomainPtr dom, > +int virDomainSetInterfaceParameters (virDomainPtr domain, This changes dom -> domain. IMHO this is no better than we started with. We should aim for consistency naming, so rather than only changing the .h, we should change .c and .h to match preferred naming. Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH 1/1] includes: function parameter names same in headers
Headers use same function parameter names as definition code. --- include/libvirt/libvirt-domain.h| 26 +- include/libvirt/libvirt-event.h | 4 ++-- include/libvirt/libvirt-host.h | 4 ++-- include/libvirt/libvirt-interface.h | 4 ++-- include/libvirt/libvirt-network.h | 6 +++--- include/libvirt/libvirt-nwfilter.h | 2 +- include/libvirt/libvirt-qemu.h | 2 +- include/libvirt/libvirt-secret.h| 4 ++-- include/libvirt/libvirt-storage.h | 12 ++-- include/libvirt/libvirt-stream.h| 22 +++--- 10 files changed, 43 insertions(+), 43 deletions(-) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index 4048acf38..60ec35d87 100644 --- a/include/libvirt/libvirt-domain.h +++ b/include/libvirt/libvirt-domain.h @@ -1088,7 +1088,7 @@ int virConnectNumOfDomains (virConnectPtr conn); /* * Get connection from domain. */ -virConnectPtr virDomainGetConnect (virDomainPtr domain); +virConnectPtr virDomainGetConnect (virDomainPtr dom); /* * Domain creation and destruction @@ -1109,7 +1109,7 @@ virDomainPtrvirDomainLookupByID (virConnectPtr conn, virDomainPtrvirDomainLookupByUUID (virConnectPtr conn, const unsigned char *uuid); virDomainPtrvirDomainLookupByUUIDString (virConnectPtr conn, - const char *uuid); + const char *uuidstr); typedef enum { VIR_DOMAIN_SHUTDOWN_DEFAULT= 0,/* hypervisor choice */ @@ -1161,11 +1161,11 @@ int virDomainFree (virDomainPtr domain); */ int virDomainSuspend(virDomainPtr domain); int virDomainResume (virDomainPtr domain); -int virDomainPMSuspendForDuration (virDomainPtr domain, +int virDomainPMSuspendForDuration (virDomainPtr dom, unsigned int target, unsigned long long duration, unsigned int flags); -int virDomainPMWakeup (virDomainPtr domain, +int virDomainPMWakeup (virDomainPtr dom, unsigned int flags); /* * Domain save/restore @@ -1626,11 +1626,11 @@ int virDomainInterfaceStats (virDomainPtr dom, */ # define VIR_DOMAIN_BANDWIDTH_OUT_BURST "outbound.burst" -int virDomainSetInterfaceParameters (virDomainPtr dom, +int virDomainSetInterfaceParameters (virDomainPtr domain, const char *device, virTypedParameterPtr params, int nparams, unsigned int flags); -int virDomainGetInterfaceParameters (virDomainPtr dom, +int virDomainGetInterfaceParameters (virDomainPtr domain, const char *device, virTypedParameterPtr params, int *nparams, unsigned int flags); @@ -1697,7 +1697,7 @@ struct _virDomainBlockInfo { * offset, similar to 'ls')*/ }; -int virDomainGetBlockInfo(virDomainPtr dom, +int virDomainGetBlockInfo(virDomainPtr domain, const char *disk, virDomainBlockInfoPtr info, unsigned int flags); @@ -1869,7 +1869,7 @@ int virDomainPinEmulator (virDomainPtr domain, unsigned int flags); int virDomainGetEmulatorPinInfo (virDomainPtr domain, - unsigned char *cpumaps, + unsigned char *cpumap, int maplen, unsigned int flags); @@ -1889,7 +1889,7 @@ struct _virDomainIOThreadInfo { void virDomainIOThreadInfoFree(virDomainIOThreadInfoPtr info); -int virDomainGetIOThreadInfo(virDomainPtr domain, +int virDomainGetIOThreadInfo(virDomainPtr dom, virDomainIOThreadInfoPtr **info, unsigned int flags);