[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