Compiling with MSVC results in the warning below: app/test-pmd/cmdline_flow.c(13964): warning C4098: 'cmd_set_raw_parsed': 'void' function returning a value
Signed-off-by: Andre Muezerie <andre...@linux.microsoft.com> Signed-off-by: Chengwen Feng <fengcheng...@huawei.com> Acked-by: Bruce Richardson <bruce.richard...@intel.com> --- app/test-pmd/cmdline_flow.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c index 24323d8891..15a18db2c7 100644 --- a/app/test-pmd/cmdline_flow.c +++ b/app/test-pmd/cmdline_flow.c @@ -13952,11 +13952,15 @@ cmd_set_raw_parsed(const struct buffer *in) int gtp_psc = -1; /* GTP PSC option index. */ const void *src_spec; - if (in->command == SET_SAMPLE_ACTIONS) - return cmd_set_raw_parsed_sample(in); + if (in->command == SET_SAMPLE_ACTIONS) { + cmd_set_raw_parsed_sample(in); + return; + } else if (in->command == SET_IPV6_EXT_PUSH || - in->command == SET_IPV6_EXT_REMOVE) - return cmd_set_ipv6_ext_parsed(in); + in->command == SET_IPV6_EXT_REMOVE) { + cmd_set_ipv6_ext_parsed(in); + return; + } RTE_ASSERT(in->command == SET_RAW_ENCAP || in->command == SET_RAW_DECAP); if (in->command == SET_RAW_ENCAP) { -- 2.48.1.vfs.0.0