Re: [ovs-dev] [PATCH V2 3/3] clang: Fix the alignment warning.

2013-07-23 Thread Alex Wang
Thanks Ben, On Tue, Jul 23, 2013 at 12:34 PM, Ben Pfaff 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 > > Thanks, I applied this.

Re: [ovs-dev] [PATCH V2 3/3] clang: Fix the alignment warning.

2013-07-23 Thread Ben Pfaff
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 Thanks, I applied this. I noticed one place with an unnecessary cast, so I folded this in: diff --gi

[ovs-dev] [PATCH V2 3/3] clang: Fix the alignment warning.

2013-07-22 Thread Alex Wang
This commit fixes the warning issued by 'clang' when pointer is casted to one with greater alignment. Signed-off-by: Alex Wang --- v1 -> v2: 1. use ofpbuf_at() instead of creating ofpbuf_forward_ref() 2. remove the unnecessary parenthesis in argument to ALIGNED_CAST() --- lib/hash.c