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 <[email protected]>
---
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 <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
