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