Attention is currently required from: flichtenheld, plaisthos, stipa. cron2 has posted comments on this change. ( http://gerrit.openvpn.net/c/openvpn/+/816?usp=email )
Change subject: dco-win: kernel notifications ...................................................................... Patch Set 20: Code-Review+1 (4 comments) Patchset: PS20: the code looks good and the tests confirm "it works". I have a few minor nits, though. File src/openvpn/dco_win.c: http://gerrit.openvpn.net/c/openvpn/+/816/comment/5d25f041_1b79d04b : PS20, Line 644: dco_handle_overlapped_success(dco_context_t *dco, bool queued) can you add a comment what this function does? As a non-windows programmer I find `Overlapped` a foreign concept File src/openvpn/mudp.c: http://gerrit.openvpn.net/c/openvpn/+/816/comment/d7e9b590_0c316b58 : PS20, Line 409: #if defined(ENABLE_DCO) && (defined(TARGET_LINUX) || defined(TARGET_FREEBSD) || defined(TARGET_WIN32)) this is a minor nitpick, but I find this line very long and hard to read now - can you wrap it? ``` #if defined(ENABLE_DCO) \ && (defined(TARGET_LINUX) || defined(TARGET_FREEBSD) || defined(TARGET_WIN32)) ``` File src/openvpn/multi.c: http://gerrit.openvpn.net/c/openvpn/+/816/comment/3e9bcdb0_2f4ec1a0 : PS20, Line 3231: #if defined(ENABLE_DCO) && (defined(TARGET_LINUX) || defined(TARGET_FREEBSD) || defined(TARGET_WIN32)) same thing -- To view, visit http://gerrit.openvpn.net/c/openvpn/+/816?usp=email To unsubscribe, or for help writing mail filters, visit http://gerrit.openvpn.net/settings Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: Iedc10616225f6769c66d3c29d4a462b622ebbc6e Gerrit-Change-Number: 816 Gerrit-PatchSet: 20 Gerrit-Owner: stipa <[email protected]> Gerrit-Reviewer: cron2 <[email protected]> Gerrit-Reviewer: flichtenheld <[email protected]> Gerrit-Reviewer: plaisthos <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-Attention: plaisthos <[email protected]> Gerrit-Attention: flichtenheld <[email protected]> Gerrit-Attention: stipa <[email protected]> Gerrit-Comment-Date: Thu, 20 Feb 2025 07:23:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
