On Thu, Mar 13, 2014 at 11:35:27AM -0700, Gurucharan Shetty wrote:
> On Thu, Mar 13, 2014 at 11:22 AM, Ben Pfaff <b...@nicira.com> wrote:
> > On Thu, Mar 13, 2014 at 09:57:12AM -0700, Gurucharan Shetty wrote:
> >> On Tue, Mar 11, 2014 at 1:56 PM, Ben Pfaff <b...@nicira.com> wrote:
> >> > This is broken out into a separate commit because it adds new MSVC
> >> > specific code and I don't have MSVC around to test whether it's correct.
> >> >
> >> > CC: Gurucharan Shetty <gshe...@nicira.com>
> >> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> >>
> >> Looks good to me.
> >>
> >> I do not see any users of it though.
> >>
> >> The idea is to do something like this, right?
> >>
> >> #define OVS_ALIGNED_STRUCT(N, TAG) __declspec(align(N)) struct TAG
> >>
> >> struct test {
> >> int a;
> >> int b;
> >> char c;
> >> };
> >> OVS_ALIGNED_STRUCT(64, test);
> >
> > The comment describes intended use:
> > * e.g. The following:
> > * OVS_ALIGNED_STRUCT(64, mystruct) { ... };
> > * is equivalent to the following except that it specifies 64-byte
> > alignment:
> > * struct mystruct { ... };
> After I saw the user, it became obvious. I compiled it with visual
> studio and it did not complain. (The documentation says it is the
> correct way too.)
Thanks for checking!
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
