On 22 June 2016 at 13:50, Sairam Venugopal <[email protected]> wrote:

> Declaration of 'event' hides previous local declaration. Rename this to
> evt. The other variable wasn't being used.
>
> Signed-off-by: Sairam Venugopal <[email protected]>
>
Applied, thanks!


> ---
>  datapath-windows/ovsext/Vport.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/datapath-windows/ovsext/Vport.c
> b/datapath-windows/ovsext/Vport.c
> index 222b2c1..9d9b367 100644
> --- a/datapath-windows/ovsext/Vport.c
> +++ b/datapath-windows/ovsext/Vport.c
> @@ -532,14 +532,14 @@ HvUpdateNic(POVS_SWITCH_CONTEXT switchContext,
>      vport->numaNodeId = nicParam->NumaNodeId;
>
>      if (nameChanged) {
> -        OVS_EVENT_ENTRY event;
> -        event.portNo = vport->portNo;
> -        event.ovsType = vport->ovsType;
> -        event.upcallPid = vport->upcallPid;
> -        RtlCopyMemory(&event.ovsName, &vport->ovsName, sizeof
> event.ovsName);
> -        event.type = OVS_EVENT_LINK_DOWN;
> +        OVS_EVENT_ENTRY evt;
> +        evt.portNo = vport->portNo;
> +        evt.ovsType = vport->ovsType;
> +        evt.upcallPid = vport->upcallPid;
> +        RtlCopyMemory(&evt.ovsName, &vport->ovsName, sizeof evt.ovsName);
> +        evt.type = OVS_EVENT_LINK_DOWN;
>          OvsRemoveAndDeleteVport(NULL, switchContext, vport, FALSE, TRUE);
> -        OvsPostEvent(&event);
> +        OvsPostEvent(&evt);
>      }
>
>      NdisReleaseRWLock(switchContext->dispatchLock, &lockState);
> --
> 2.5.0.windows.1
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to