ACK, thanks, plus
20:39 <@dazo> cron2: you can add ACK on me too on that one, I believe it is the
same we've already reviewed
"Stared at code", plus "passed all my client tests" plus "adds cmocka
tests which are really useful for that stuff" (and they pass :) ).
.. and this is a real improvement to the code - it just removes lines,
and intermediate variables. Too much magic, that is not needed.
Your patch has been applied to the master and release/2.4 branch.
Backporting to release/2.3 is not a trivial matter of "there is whitespace",
but it seems infrastructure is missing (CRYPT_ERROR()), and the grand
unified reformatting is too much for git to grok the differences in
context. It should be fairly easy to do by hand, though.
commit a87e1431baccd49a9344cfc63ab7446c4317fa2f (master)
commit 653d39192297880b4fbc6a53cce6f8692782e1b7 (release/2.4)
Author: Steffan Karger
Date: Fri May 5 19:44:51 2017 +0200
cleanup: merge packet_id_alloc_outgoing() into packet_id_write()
Signed-off-by: Steffan Karger <[email protected]>
Acked-by: Gert Doering <[email protected]>
Acked-by: David Sommerseth <[email protected]>
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg14541.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
