[ovs-dev] [minor cleanups 1/5] ofproto: Remove write-only variable.

[Ethan Jackson]

Looks good, thanks.

Ethan

On Sun, May 13, 2012 at 4:58 PM, Ben Pfaff <blp at nicira.com> wrote:
> Found by clang.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
>  ofproto/ofproto.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index 98bb912..60dd36a 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -3227,14 +3227,13 @@ handle_nxt_set_controller_id(struct ofconn *ofconn,
>  static enum ofperr
>  handle_barrier_request(struct ofconn *ofconn, const struct ofp_header *oh)
>  {
> -    struct ofp_header *ob;
>     struct ofpbuf *buf;
>
>     if (ofconn_has_pending_opgroups(ofconn)) {
>         return OFPROTO_POSTPONE;
>     }
>
> -    ob = make_openflow_xid(sizeof *ob, OFPT10_BARRIER_REPLY, oh->xid, &buf);
> +    make_openflow_xid(sizeof *oh, OFPT10_BARRIER_REPLY, oh->xid, &buf);
>     ofconn_send_reply(ofconn, buf);
>     return 0;
>  }
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev