I'm not a great fan of patches that do nothing more than "appease compilers",
and some of the conversions should really not be necessary - OTOH,
compilers have become better at spotting implicit conversions that *are*
not intended, losing precision and breaking things in the long run - so
better be explicit about int types...
I have stared-at-code (in addition to the ACK by Arne, recorded in Gerrit,
which somehow did not make it into the mail) - looks good. Also, gave
it some basic tests on Linux and FreeBSD + GHA (-Werror builds), and
all fine..
Your patch has been applied to the master branch.
commit 53449cb61ff569c4862926c7999d50f634030fd9
Author: Frank Lichtenheld
Date: Tue Sep 10 14:20:08 2024 +0200
Various fixes for -Wconversion errors
Signed-off-by: Frank Lichtenheld <fr...@lichtenheld.com>
Acked-by: Arne Schwabe <a...@rfc2549.org>
Message-Id: <20240910122008.23507-1-g...@greenie.muc.de>
URL:
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg29172.html
Signed-off-by: Gert Doering <g...@greenie.muc.de>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
