[M] Change in openvpn[master]: packet_id: Avoid conversion warnings

"razvanc \(Code Review\) via Openvpn-devel" <[email protected]>
Newsgroups gmane.network.openvpn.devel
Message-ID <7194360f827a86c453e21917a0c3a0ba89a290cd-EmailReviewComments-HTML@gerrit.openvpn.net>
Attention is currently required from: flichtenheld.

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

Change subject: packet_id: Avoid conversion warnings
......................................................................


Patch Set 8:

(1 comment)

File src/openvpn/circ_list.h:

http://gerrit.openvpn.net/c/openvpn/+/1407/comment/5a2c3702_25dbc08b?usp=email :
PS8, Line 63:         const size_t so = sizeof(list_type) + sizeof((dest)->x_list[0]) * (size); \
> maybe then we could just have an explicit cast to size_t here?
FWIW that was just an observation on my part, not a big deal. We will probably do the work of removing `-Wsign-conversion` warnings as well at some point, and this can be done then if no other changes are requested now other than this.



-- 
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1407?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: Ib0584e8728701cca10ac5675c9cb0e6f5eb901ac
Gerrit-Change-Number: 1407
Gerrit-PatchSet: 8
Gerrit-Owner: flichtenheld <[email protected]>
Gerrit-Reviewer: plaisthos <[email protected]>
Gerrit-Reviewer: razvanc <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: flichtenheld <[email protected]>
Gerrit-Comment-Date: Tue, 18 Aug 2026 15:14:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: plaisthos <[email protected]>
Comment-In-Reply-To: razvanc <[email protected]>

_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.