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

I can't claim to understand the MSVC intricacies, but the explanation
is good, and it shouldn't break non-MSVC compiles.  The dangling ";" 
after an #endif is truly OpenVPN-style ugly, though...  we avoid
it elsewhere by defining

  #define __attribute__(x)

for MSVC builds, but error.h is special... the code would benefit from
a comment, though.

Your patch has been applied to the master branch.

commit ad2140e0337f99c183e732b61df03ae29bdec766
Author: Arne Schwabe
Date:   Wed May 12 15:15:08 2021 +0200

     Add noreturn attribute for MSVC to assert_failed method.

     Signed-off-by: Arne Schwabe <a...@rfc2549.org>
     Acked-by: Gert Doering <g...@greenie.muc.de>
     Message-Id: <20210512131511.1309914-7-a...@rfc2549.org>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg22344.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