Its correct as written. the && takes precidence over the || and the statement gets evaluated in the order i intended. i will add the parentheses to quiet the warning when i get a chance
> On Jul 3, 2016, at 9:01 AM, Ralph Castain <r...@open-mpi.org> wrote: > > I agree with the compiler - I can’t figure out exactly what was meant here > either: > > bml_r2.c: In function ‘mca_bml_r2_endpoint_add_btl’: > bml_r2.c:271:21: warning: suggest parentheses around ‘&&’ within ‘||’ > [-Wparentheses] > if ((btl_in_use && (btl_flags & MCA_BTL_FLAGS_RDMA) || > ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > _______________________________________________ > devel mailing list > de...@open-mpi.org > Subscription: https://www.open-mpi.org/mailman/listinfo.cgi/devel > Link to this post: > http://www.open-mpi.org/community/lists/devel/2016/07/19150.php