Re: [Dolfin] [Branch ~dolfin-core/dolfin/main] Rev 5731: Remove parts of pointer/reference interface in Form. Prefer shared_ptr interfaces instead.

[Marie E. Rognes]

On 03/06/2011 09:30 PM, nore...@launchpad.net wrote:
    if (parameters["max_dimension"].change_count()>  0
        &&  V.dim()>  max_dimension)
+  {
      return true;
-
-  // Otherwise, not done.
-  return false;
+  }
+  else
+    return false;
  }


I notice that my early returns keep getting moved into else clauses... I 
find this approach less readable, especially when there are nested ifs. 
Why is it the preferred way?

--
Marie

_______________________________________________
Mailing list: https://launchpad.net/~dolfin
Post to     : dolfin@lists.launchpad.net
Unsubscribe : https://launchpad.net/~dolfin
More help   : https://help.launchpad.net/ListHelp