On Tue, Aug 20, 2013 at 03:12:16PM -0700, Ben Pfaff wrote:
> On Tue, Aug 20, 2013 at 05:05:26PM -0400, Ed Maste wrote:
> > On 20 August 2013 14:10, Ben Pfaff <b...@nicira.com> wrote:
> > > Without this patch, I get Clang warnings that I don't understand for each
> > > of the non-static functions in dirs.c:
> > >
> > >     warning: control reaches end of non-void function
> > 
> > This is a clang bug; in your example the call to bar() is omitted by
> > clang, and the warning is thus actually true.  I encountered the bug
> > while looking at other issues in FreeBSD's stdatomic.h and my
> > colleague distilled a simple test case and submitted a bug report[1]
> > for it yesterday.  As of this morning it's fixed in clang svn[2].
> 
> Thanks, that's good to know.  I'm using a Clang nightly build, so I'll
> upgrade tomorrow and see if it fixes the problem for me, and I'll plan
> to drop this patch.

The new nightly fixes this problem but introduces new -Wthread-safety
warnings.  So, I'm going to drop this patch and go look at those
warnings.

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to