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

Reply via email to