Acked-by: Gert Doering <[email protected]>
Thanks :-) - given that this is somewhat trivial, I have not actually
run a binary to look at the messages. I have counted arguments and done
a test build to see if new warnings show up (no).
I *do* see an old warning...
tun.c: In function 'windows_set_mtu':
tun.c:5523:21: warning: format '%u' expects argument of type 'unsigned int',
but argument 5 has type 'DWORD {aka long unsigned int}' [-Wformat=]
msg(M_WARN, "TUN: Setting %s mtu failed: %s [status=%u if_index=%d]",
mmmh...
Your patch has been applied to the master branch.
commit efe01d52e36c597484b6fa24c4820b6345d08ae6
Author: Christopher Schenk
Date: Tue Jun 30 11:54:44 2020 +0200
Unified success messages for setting mtu
Acked-by: Gert Doering <[email protected]>
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg20171.html
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