From: Vratko Polak Date: Thu, 30 Nov 2023 16:42:25 +0000 (+0100) Subject: dev: use the endian reply macro X-Git-Tag: v24.06-rc0~106 X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=3a9c3ebcc1a818031efdf83d552bfb21206d2eae;p=vpp.git dev: use the endian reply macro Checkstyle also forces the new indentation. Type: fix Fixes: ddf6cec37027547ff7cc61e15bb8080664d41514 Change-Id: Ife96928d6ca30ba94e1c423d557d6ed9d68eca2b Signed-off-by: Vratko Polak --- diff --git a/src/vnet/dev/dev_api.c b/src/vnet/dev/dev_api.c index 4161c3211bd..db6d7b0b206 100644 --- a/src/vnet/dev/dev_api.c +++ b/src/vnet/dev/dev_api.c @@ -54,17 +54,17 @@ vl_api_dev_attach_t_handler (vl_api_dev_attach_t *mp) vec_free (a.args); - REPLY_MACRO3 (VL_API_DEV_ATTACH_REPLY, vec_len (error_string), ({ - rmp->retval = rv; - if (error_string) - { - rmp->dev_index = ~0; - vl_api_vec_to_api_string (error_string, - &rmp->error_string); - } - else - rmp->dev_index = a.dev_index; - })); + REPLY_MACRO3_END (VL_API_DEV_ATTACH_REPLY, vec_len (error_string), ({ + rmp->retval = rv; + if (error_string) + { + rmp->dev_index = ~0; + vl_api_vec_to_api_string (error_string, + &rmp->error_string); + } + else + rmp->dev_index = a.dev_index; + })); vec_free (a.args); vec_free (error_string); @@ -86,12 +86,12 @@ vl_api_dev_detach_t_handler (vl_api_dev_detach_t *mp) if (rv != VNET_DEV_OK) error_string = format (0, "%U", format_vnet_dev_rv, rv); - REPLY_MACRO3 (VL_API_DEV_DETACH_REPLY, vec_len (error_string), ({ - rmp->retval = rv; - if (error_string) - vl_api_vec_to_api_string (error_string, - &rmp->error_string); - })); + REPLY_MACRO3_END (VL_API_DEV_DETACH_REPLY, vec_len (error_string), ({ + rmp->retval = rv; + if (error_string) + vl_api_vec_to_api_string (error_string, + &rmp->error_string); + })); vec_free (error_string); } @@ -128,17 +128,17 @@ vl_api_dev_create_port_if_t_handler (vl_api_dev_create_port_if_t *mp) vec_free (a.args); - REPLY_MACRO3 (VL_API_DEV_CREATE_PORT_IF_REPLY, vec_len (error_string), ({ - rmp->retval = rv; - if (error_string) - { - rmp->sw_if_index = ~0; - vl_api_vec_to_api_string (error_string, - &rmp->error_string); - } - else - rmp->sw_if_index = a.sw_if_index; - })); + REPLY_MACRO3_END (VL_API_DEV_CREATE_PORT_IF_REPLY, vec_len (error_string), ({ + rmp->retval = rv; + if (error_string) + { + rmp->sw_if_index = ~0; + vl_api_vec_to_api_string (error_string, + &rmp->error_string); + } + else + rmp->sw_if_index = a.sw_if_index; + })); vec_free (a.args); vec_free (error_string); @@ -160,12 +160,12 @@ vl_api_dev_remove_port_if_t_handler (vl_api_dev_remove_port_if_t *mp) if (rv != VNET_DEV_OK) error_string = format (0, "%U", format_vnet_dev_rv, rv); - REPLY_MACRO3 (VL_API_DEV_REMOVE_PORT_IF_REPLY, vec_len (error_string), ({ - rmp->retval = rv; - if (error_string) - vl_api_vec_to_api_string (error_string, - &rmp->error_string); - })); + REPLY_MACRO3_END (VL_API_DEV_REMOVE_PORT_IF_REPLY, vec_len (error_string), ({ + rmp->retval = rv; + if (error_string) + vl_api_vec_to_api_string (error_string, + &rmp->error_string); + })); vec_free (error_string); }