Attention is currently required from: Arne Schwabe, Frank Lichtenheld. Răzvan Cojocaru 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: (1 comment) File src/openvpn/buffer.h: https://gerrit.openvpn.net/c/openvpn/+/1997/comment/601e60de_efe382ea?usp=email : PS1, Line 795: * @return true if the buffer was changed, i.e. at least one character was appended. > I don't understand this one I was thinking of the empty string: `buf_puts(buf, "")` with `cap > 1` returns `true`, but nothing gets appended and the buffer is unchanged. So true really means "there was room to append", not "something was appended". -- 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: Arne Schwabe <[email protected]> Gerrit-Attention: Frank Lichtenheld <[email protected]> Gerrit-Comment-Date: Wed, 14 Oct 2026 11:01:31 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Răzvan Cojocaru <[email protected]> Comment-In-Reply-To: Frank Lichtenheld <[email protected]>
_______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
