Browse code

vlan: Remove -Wconversion override

Only Debian 11 (GCC 10) complains about this.
We decided not to add work-arounds for GCC 10
and older and instead accept that they do not
build -Werror clean.

Change-Id: I73c46ac630834a8cf8894aaa2dcc429fbedd3db7
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1179
Message-Id: <20251008105507.9997-1-gert@greenie.muc.de>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg33285.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>

Frank Lichtenheld authored on 2025/10/08 19:55:01
Showing 1 changed files
... ...
@@ -43,11 +43,6 @@ vlanhdr_get_vid(const struct openvpn_8021qhdr *hdr)
43 43
     return ntohs(hdr->pcp_cfi_vid & OPENVPN_8021Q_MASK_VID);
44 44
 }
45 45
 
46
-#if defined(__GNUC__) || defined(__clang__)
47
-#pragma GCC diagnostic push
48
-#pragma GCC diagnostic ignored "-Wconversion"
49
-#endif
50
-
51 46
 /*
52 47
  * Set the VLAN Identifier (VID) in an IEEE 802.1Q header.
53 48
  *
... ...
@@ -61,10 +56,6 @@ vlanhdr_set_vid(struct openvpn_8021qhdr *hdr, const uint16_t vid)
61 61
         (hdr->pcp_cfi_vid & ~OPENVPN_8021Q_MASK_VID) | (htons(vid) & OPENVPN_8021Q_MASK_VID);
62 62
 }
63 63
 
64
-#if defined(__GNUC__) || defined(__clang__)
65
-#pragma GCC diagnostic pop
66
-#endif
67
-
68 64
 /*
69 65
  * vlan_decapsulate - remove 802.1q header and return VID
70 66
  *