As mentioned every now and then, I'm not a big fan of patches that move code around - we frequently use "git blame" to figure out why certain code pieces came to be, and this gets more annoying if code gets moved. This said, *if* code needs to move, I do prefer a patch that has "no other changes" so "--color-moved=zebra" can visually prove that "nothing has changed". In this case, one "static" appeared, but nothing else :-)
Test compiled, as usual. Your patch has been applied to the master branch. commit 7bcafb316ecea68b1acda9e137df9dce18afcc8c Author: Antonio Quartulli Date: Wed Jul 23 08:07:41 2025 +0200 dco_linux: rearrange functions Signed-off-by: Antonio Quartulli <anto...@mandelbit.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20250723060747.19524-1-g...@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg32263.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