Thanks Ben,
On Tue, Jul 23, 2013 at 12:34 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Mon, Jul 22, 2013 at 03:47:19PM -0700, Alex Wang wrote:
> > This commit fixes the warning issued by 'clang' when pointer is casted
> > to one with greater alignment.
> >
> > Signed-off-by: Alex Wang <al...@nicira.com>
>
> Thanks, I applied this.
>
> I noticed one place with an unnecessary cast, so I folded this in:
>
> diff --git a/lib/netlink.c b/lib/netlink.c
> index 9ed925b..50444ab 100644
> --- a/lib/netlink.c
> +++ b/lib/netlink.c
> @@ -711,8 +711,7 @@ nl_policy_parse(const struct ofpbuf *msg, size_t
> nla_offset,
> return false;
> }
>
> - NL_ATTR_FOR_EACH (nla, left,
> - (struct nlattr *) ofpbuf_at(msg, nla_offset, 0),
> + NL_ATTR_FOR_EACH (nla, left, ofpbuf_at(msg, nla_offset, 0),
> msg->size - nla_offset)
> {
> uint16_t type = nl_attr_type(nla);
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
