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