build: remove duplicate checks for cflags
authorBruce Richardson <bruce.richardson@intel.com>
Thu, 19 Jul 2018 14:22:06 +0000 (15:22 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 26 Jul 2018 08:09:20 +0000 (10:09 +0200)
There is no need to check for each library, driver and example whether
certain cflags are supported. Instead of checking inside the loop, do
so outside and reuse the value.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
drivers/meson.build
examples/meson.build
lib/meson.build

index 27d66cf..f94e2fe 100644 (file)
@@ -11,6 +11,10 @@ driver_classes = ['common',
               'event',   # depends on common, bus, mempool and net.
               'raw']     # depends on common, bus, mempool, net and event.
 
+default_cflags = machine_args
+if cc.has_argument('-Wno-format-truncation')
+       default_cflags += '-Wno-format-truncation'
+endif
 foreach class:driver_classes
        drivers = []
        std_deps = []
@@ -31,10 +35,7 @@ foreach class:driver_classes
                allow_experimental_apis = false
                sources = []
                objs = []
-               cflags = machine_args
-               if cc.has_argument('-Wno-format-truncation')
-                       cflags += '-Wno-format-truncation'
-               endif
+               cflags = default_cflags
                includes = [include_directories(drv_path)]
                # set up internal deps. Drivers can append/override as necessary
                deps = std_deps
index e655887..4ee7a11 100644 (file)
@@ -18,15 +18,16 @@ else
        examples = get_option('examples').split(',')
        allow_skips = false # error out if we can't build a requested app
 endif
+default_cflags = machine_args
+if cc.has_argument('-Wno-format-truncation')
+       default_cflags += '-Wno-format-truncation'
+endif
 foreach example: examples
        name = example
        build = true
        sources = []
        allow_experimental_apis = false
-       cflags = machine_args
-       if cc.has_argument('-Wno-format-truncation')
-               cflags += '-Wno-format-truncation'
-       endif
+       cflags = default_cflags
 
        ext_deps = [execinfo]
        includes = [include_directories(example)]
index eeac10e..eb91f10 100644 (file)
@@ -26,6 +26,10 @@ libraries = [ 'compat', # just a header, used for versioning
        # flow_classify lib depends on pkt framework table lib
        'flow_classify', 'bpf']
 
+default_cflags = machine_args
+if cc.has_argument('-Wno-format-truncation')
+       default_cflags += '-Wno-format-truncation'
+endif
 foreach l:libraries
        build = true
        name = l
@@ -34,11 +38,7 @@ foreach l:libraries
        sources = []
        headers = []
        includes = []
-       cflags = machine_args
-       if cc.has_argument('-Wno-format-truncation')
-           cflags += '-Wno-format-truncation'
-       endif
-
+       cflags = default_cflags
        objs = [] # other object files to link against, used e.g. for
                  # instruction-set optimized versions of code