[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 <3f8c5ece87d62b1012065fbbc13c7abf6cb185f6-EmailReviewComments-HTML@gerrit.openvpn.net>
Attention is currently required from: flichtenheld, plaisthos.

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:

(2 comments)

File src/openvpn/circ_list.h:

http://gerrit.openvpn.net/c/openvpn/+/1407/comment/bf558a9d_7044c752?usp=email :
PS8, Line 63:         const size_t so = sizeof(list_type) + sizeof((dest)->x_list[0]) * (size); \
Strictly speaking, if `size` is `int` this still breaks `-Wsign-conversion`, but we're using `-Wno-sign-conversion` today, so it should be fine for now.


File src/openvpn/packet_id.c:

http://gerrit.openvpn.net/c/openvpn/+/1407/comment/01de02bf_b763fbda?usp=email :
PS8, Line 658: 
Accidental newline insert?



-- 
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-CC: openvpn-devel <[email protected]>
Gerrit-CC: razvanc <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
Gerrit-Attention: flichtenheld <[email protected]>
Gerrit-Comment-Date: Mon, 17 Aug 2026 18:43:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

_______________________________________________
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.