This all makes sense - thanks for spotting. No idea what we were looking at when merging the original patch...
Not sure introducing the extra define improves readability (the old context made it quite clear that "1600" is not "a random number out of thin air") - but since TUN_MTU_DEFAULT has a define in mtu.h, why not. Your patch has been applied to the master branch. commit 5c4744f28e2adb3fc8f6fb3b3c8ffe22636eb0a0 Author: Frank Lichtenheld Date: Sat Aug 23 17:36:46 2025 +0200 Clean up documentation for --tun-mtu-max Signed-off-by: Frank Lichtenheld <fr...@lichtenheld.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20250823153652.30938-1-g...@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg32663.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