[ovs-dev] [PATCH 13/13] ofproto-dpif-xlate: Fix dead assignment reported by clang.
Greg Rose
gvrose8192 at gmail.com
Fri Sep 15 17:42:34 UTC 2017
On 09/08/2017 10:59 AM, Bhanuprakash Bodireddy wrote:
> Clang reports that value stored in to ac_offset is never read in the
> function.
>
> Signed-off-by: Bhanuprakash Bodireddy <bhanuprakash.bodireddy at intel.com>
> ---
> ofproto/ofproto-dpif-xlate.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
> index 9e1f837..5d83666 100644
> --- a/ofproto/ofproto-dpif-xlate.c
> +++ b/ofproto/ofproto-dpif-xlate.c
> @@ -5370,7 +5370,6 @@ compose_clone(struct xlate_ctx *ctx, const struct ofpact_nest *oc)
> if (ctx->xbridge->support.clone) { /* Use clone action */
> /* Use clone action as datapath clone. */
> offset = nl_msg_start_nested(ctx->odp_actions, OVS_ACTION_ATTR_CLONE);
> - ac_offset = ctx->odp_actions->size;
> do_xlate_actions(oc->actions, oc_actions_len, ctx);
> nl_msg_end_non_empty_nested(ctx->odp_actions, offset);
> goto dp_clone_done;
>
Not in this section of code anyway.
Looks good.
Reviewed-by: Greg Rose <gvrose8192 at gmail.com>
More information about the dev
mailing list