Attention is currently required from: plaisthos. flichtenheld has posted comments on this change. ( http://gerrit.openvpn.net/c/openvpn/+/1191?usp=email )
Change subject: Allow route_ipv6_match_host to be used outside of route.c ...................................................................... Patch Set 1: (2 comments) File src/openvpn/route.c: http://gerrit.openvpn.net/c/openvpn/+/1191/comment/1fc5a8a3_d7b71b82 : PS1, Line 713: /** Why make this doxygen? The doxygen part is covered by the comment at the declaration, isn't it? http://gerrit.openvpn.net/c/openvpn/+/1191/comment/c57f9efb_cf95f1a2 : PS1, Line 740: unsigned mask = 0xff << (8 - bits); any particular reason you left the "int" off here but used in the other places? -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/1191?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: Ief1495b52ea81cac35d78e40264372d3869423f1 Gerrit-Change-Number: 1191 Gerrit-PatchSet: 1 Gerrit-Owner: plaisthos <[email protected]> Gerrit-Reviewer: flichtenheld <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-Attention: plaisthos <[email protected]> Gerrit-Comment-Date: Tue, 16 Sep 2025 11:50:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
