Acked-by: Gert Doering <g...@greenie.muc.de>

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 <g...@greenie.muc.de>
     Message-Id: <20200630095443.7188-1-csch...@mail.uni-paderborn.de>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20171.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

Reply via email to