On 10/22/2015 04:24 PM, Nathan Sidwell wrote:
+      else
+    { /* Parent will skip this parallel itself.  */ }

Here too - actually no need to have an empty else at all.

I wanted somewhere clear for the comment to go. (Actually, I think this
is the one the compiler warns about -- empty dangling else).

Don't recall such a warning, but that doesn't mean it isn't there. About the comment, it could either go just after the if, or above it, along the lines of
 /* We handle cases X, Y, Z here.  If none of them apply, then
    the parent will skip this parallel itself.  */

In this particular loop you could also use continue; for the empty if conditions.


Bernd

Reply via email to