Stared at the changes, some are "huh, wat?" (like the const <-> static
reordering, but so what), one [[fallthrough]] was obviously forgotten
in commit fed37d0, and the struct assignment gets easier to read this
way, so double benefit :-) - wrt the CMakeFile change, GHA is fine with
it, and I don't have to understand it.

Your patch has been applied to the master branch.

commit 45eb7e1807060a927ff805b0d01fbb8cfbb772c8 (master)
Author: Frank Lichtenheld
Date:   Tue Sep 23 16:08:48 2025 +0200

     Enable a subset of -Wextra

     Signed-off-by: Frank Lichtenheld <[email protected]>
     Acked-by: Gert Doering <[email protected]>
     Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1202
     Message-Id: <[email protected]>
     URL: https://sourceforge.net/p/openvpn/mailman/message/59237558/
     Signed-off-by: Gert Doering <[email protected]>


--
kind regards,

Gert Doering



_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to