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 <[email protected]>
     Acked-by: Gert Doering <[email protected]>
     Message-Id: <[email protected]>
     URL: 
https://www.mail-archive.com/[email protected]/msg32263.html
     Signed-off-by: Gert Doering <[email protected]>


--
kind regards,

Gert Doering



_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to