[ovs-dev] [classifier-opt 04/28] ofproto-dpif: Remove superfluous cast.

Ethan Jackson ethan at nicira.com
Fri Jul 27 20:26:48 UTC 2012


Looks good thanks.

Ethan

On Fri, Jul 20, 2012 at 4:25 PM, Ben Pfaff <blp at nicira.com> wrote:
> 'packet' is already a struct ofpbuf *.
> ---
>  ofproto/ofproto-dpif.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> index dc15c15..6bc2cf4 100644
> --- a/ofproto/ofproto-dpif.c
> +++ b/ofproto/ofproto-dpif.c
> @@ -4725,7 +4725,7 @@ send_packet(const struct ofport_dpif *ofport, struct ofpbuf *packet)
>      struct flow flow;
>      int error;
>
> -    flow_extract((struct ofpbuf *) packet, 0, 0, 0, &flow);
> +    flow_extract(packet, 0, 0, 0, &flow);
>      odp_port = vsp_realdev_to_vlandev(ofproto, ofport->odp_port,
>                                        flow.vlan_tci);
>      if (odp_port != ofport->odp_port) {
> --
> 1.7.2.5
>



More information about the dev mailing list