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
Acked-by: Gert Doering
Acked-by: David Sommerseth
Message-Id: <1494006291-3522-1-git-send-email-steffan.kar...@fox-it.com>
URL:
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg14541.html
Signed-off-by: Gert Doering
--
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
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel