I agree Ben... I was thinking the same thing.  I'll get it done.

Derek


On Thu, Jul 11, 2013 at 1:35 PM, Benjamin S. Kirk
<notifications@github.com>wrote:

> In src/mesh/mesh_tools.C:
>
> > @@ -77,7 +77,7 @@ namespace {
> >
> >  // If we don't have threads we never need a join, and icpc yells a
> >  // warning if it sees an anonymous function that's never used
> > -#ifdef LIBMESH_HAVE_TBB_API
> > +#if defined(LIBMESH_HAVE_PTHREAD) || defined(LIBMESH_HAVE_TBB_API)
>
> syntactic sugar, but important going forward I think: could we also define
> a common macro, e.g.
>
> #ifdef LIBMESH_HAVE_THREADS
>
> that for the moment does exactly this? This way developers don't need to
> remember that there are multiple thread implementations behind the scenes?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/libMesh/libmesh/pull/109/files#r5150312>
> .
>