[v14,57/81] lib/node: add compile warning about use of VLAs

Message ID 1736540620-21764-58-git-send-email-andremue@linux.microsoft.com (mailing list archive)
State Superseded, archived
Delegated to: David Marchand
Headers
Series remove use of VLAs for Windows |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Andre Muezerie Jan. 10, 2025, 8:23 p.m. UTC
MSVC does not support VLAs, so we want to prevent VLAs from being
introduced under this path.

Signed-off-by: Andre Muezerie <andremue@linux.microsoft.com>
---
 lib/node/meson.build | 8 ++++++++
 1 file changed, 8 insertions(+)
  

Patch

diff --git a/lib/node/meson.build b/lib/node/meson.build
index 0bed97a96c..4c7361a53a 100644
--- a/lib/node/meson.build
+++ b/lib/node/meson.build
@@ -35,3 +35,11 @@  headers = files(
 # Strict-aliasing rules are violated by uint8_t[] to context size casts.
 cflags += '-fno-strict-aliasing'
 deps += ['graph', 'mbuf', 'lpm', 'ethdev', 'mempool', 'cryptodev', 'ip_frag']
+
+extra_flags = ['-Wvla']
+
+foreach arg: extra_flags
+    if cc.has_argument(arg)
+        cflags += arg
+    endif
+endforeach