Re: [Openvpn-devel] [PATCH] tun.h: change tun_set() return value type to void
On 06/09/2019 17:40, Lev Stipakov wrote: > This function's return value is never used, so make it void. > > Signed-off-by: Lev Stipakov > --- > src/openvpn/tun.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/src/openvpn/tun.h b/src/openvpn/tun.h > index 69831c4..475643a 100644 > --- a/src/openvpn/tun.h > +++ b/src/openvpn/tun.h > @@ -513,7 +513,7 @@ tun_event_handle(const struct tuntap *tt) > #endif > } > > -static inline unsigned int > +static inline void > tun_set(struct tuntap *tt, > struct event_set *es, > unsigned int rwflags, I did some check into the history of this function, even going into the openvpn-historical-cvs git repository. The return value of this function was defined as unsigned int since the very beginning - going back to OpenVPN 2.0_beta18 - which basically was the first public commit after the 1.6.0 release. In my very quick look, not even once has the return value been used since that time. So this change makes sense. No need to have a return value which has not been interesting since November 2004. Acked-By: David Sommerseth -- kind regards, David Sommerseth OpenVPN Inc ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
Re: [Openvpn-devel] [PATCH] tun.h: change tun_set() return value type to void
Hi, On 06/09/2019 17:40, Lev Stipakov wrote: > This function's return value is never used, so make it void. > > Signed-off-by: Lev Stipakov Does what it says and git suggests that its return value was never used. Using void as return type also clarifies its usage. Acked-by: Antonio Quartulli -- Antonio Quartulli ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
[Openvpn-devel] [PATCH] tun.h: change tun_set() return value type to void
This function's return value is never used, so make it void. Signed-off-by: Lev Stipakov --- src/openvpn/tun.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/openvpn/tun.h b/src/openvpn/tun.h index 69831c4..475643a 100644 --- a/src/openvpn/tun.h +++ b/src/openvpn/tun.h @@ -513,7 +513,7 @@ tun_event_handle(const struct tuntap *tt) #endif } -static inline unsigned int +static inline void tun_set(struct tuntap *tt, struct event_set *es, unsigned int rwflags, @@ -539,7 +539,6 @@ tun_set(struct tuntap *tt, #endif tt->rwflags_debug = rwflags; } -return rwflags; } const char *tun_stat(const struct tuntap *tt, unsigned int rwflags, struct gc_arena *gc); -- 2.7.4 ___ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel