Attention is currently required from: Arne Schwabe, Răzvan Cojocaru. Frank Lichtenheld has posted comments on this change by Frank Lichtenheld. ( https://gerrit.openvpn.net/c/openvpn/+/1997?usp=email )
Change subject: buffer: Fix small issues buf_printf and buf_puts ...................................................................... Patch Set 1: (2 comments) File src/openvpn/buffer.h: https://gerrit.openvpn.net/c/openvpn/+/1997/comment/09b6961e_a7004139?usp=email : PS1, Line 795: * @return true if the buffer was changed, i.e. at least one character was appended. > `buf_puts(buf, "")` returns `true` whenever `cap > 1` so that's not > technically true. […] I don't understand this one File tests/unit_tests/openvpn/test_buffer.c: https://gerrit.openvpn.net/c/openvpn/+/1997/comment/16d53d2a_2ec6f17a?usp=email : PS1, Line 68: assert_buf_equals_str(&buf, "123some text, t"); > This passes with the old code as well. […] Yes, this was just to confirm the behavior. It did not change. -- To view, visit https://gerrit.openvpn.net/c/openvpn/+/1997?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.openvpn.net/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: openvpn Gerrit-Branch: master Gerrit-Change-Id: I1d6ed5d40910359b65b9f5c5d8f56ad3d3bc827a Gerrit-Change-Number: 1997 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Lichtenheld <[email protected]> Gerrit-Reviewer: Arne Schwabe <[email protected]> Gerrit-Reviewer: Răzvan Cojocaru <[email protected]> Gerrit-CC: openvpn-devel <[email protected]> Gerrit-Attention: Răzvan Cojocaru <[email protected]> Gerrit-Attention: Arne Schwabe <[email protected]> Gerrit-Comment-Date: Wed, 14 Oct 2026 10:50:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Răzvan Cojocaru <[email protected]>
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
