Attention is currently required from: plaisthos.

flichtenheld has posted comments on this change by flichtenheld. ( 
http://gerrit.openvpn.net/c/openvpn/+/1610?usp=email )

Change subject: Remove various unused functions
......................................................................


Patch Set 12:

(1 comment)

File src/openvpn/buffer.h:

http://gerrit.openvpn.net/c/openvpn/+/1610/comment/24f5b44d_2a60a31f?usp=email :
PS12, Line 818: {
> I understand that this might not be used but feels a bit weird to remove it 
> nonetheless.
Yeah, there are some examples in here where we remove parts of an API that are 
not currently used but make sense to make the API complete and consistent. I 
don't have a strong opinion here. When you make a list of all removals that you 
think are dubious I could split the patch.



--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1610?usp=email
To unsubscribe, or for help writing mail filters, visit 
http://gerrit.openvpn.net/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: I82a4e82ff876fab715e052de1903c65b835360b5
Gerrit-Change-Number: 1610
Gerrit-PatchSet: 12
Gerrit-Owner: flichtenheld <[email protected]>
Gerrit-Reviewer: plaisthos <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
Gerrit-Comment-Date: Mon, 10 Aug 2026 12:04:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: plaisthos <[email protected]>
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to