On Mon, Apr 29, 2013 at 3:23 PM, Ben Pfaff <b...@nicira.com> wrote:

> I don't see any reason for this variable to be static.
>
Looks good to me, thanks.

>
> CC: Ethan Jackson <et...@nicira.com>
> Signed-off-by: Ben Pfaff <b...@nicira.com>
> ---
>  lib/rtnetlink-link.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/rtnetlink-link.c b/lib/rtnetlink-link.c
> index 0b3710f..459e485 100644
> --- a/lib/rtnetlink-link.c
> +++ b/lib/rtnetlink-link.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright (c) 2009, 2010 Nicira, Inc.
> + * Copyright (c) 2009, 2010, 2013 Nicira, Inc.
>   *
>   * Licensed under the Apache License, Version 2.0 (the "License");
>   * you may not use this file except in compliance with the License.
> @@ -49,7 +49,7 @@ rtnetlink_link_parse(struct ofpbuf *buf,
>          [IFLA_ADDRESS] = { .type = NL_A_UNSPEC, .optional = true },
>      };
>
> -    static struct nlattr *attrs[ARRAY_SIZE(policy)];
> +    struct nlattr *attrs[ARRAY_SIZE(policy)];
>
>      parsed = nl_policy_parse(buf, NLMSG_HDRLEN + sizeof(struct ifinfomsg),
>                               policy, attrs, ARRAY_SIZE(policy));
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to