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 <lstipa...@gmail.com> > --- > 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 <dav...@openvpn.net> -- kind regards, David Sommerseth OpenVPN Inc _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
