THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#322 - remove superfluous "Dumux::"
User who did this - Bernd Flemisch (bernd)

----------
Recently, we merged
https://git.iws.uni-stuttgart.de/dumux-repositories/dumux/merge_requests/65

This change affected 138 files. It removed the "Dune::" from the "elements" and 
"intersections" functions. The only argument given was that the compiler is 
able to do it. While I liked that change, it is much more obscure than the one 
proposed now, since we now call a function from within a namespace "Dumux" that 
is out of another namespace "Dune" without qualifying it like this.

I really see no reason why I should use the namespace qualifier if I want to 
use a name from the very namespace that I'm in, apart from avoiding technical 
ambiguities. We also never say

namespace Dumux {
...
class LensProblem : public Dumux::ImplicitPorousMediaProblem
...
}

----------

More information can be found at the following URL:
http://www.dumux.org/flyspray/index.php?do=details&task_id=322#comment690

You are receiving this message because you have requested it from the Flyspray 
bugtracking system.  If you did not expect this message or don't want to 
receive mails in future, you can change your notification settings at the URL 
shown above.
_______________________________________________
Dumux mailing list
[email protected]
https://listserv.uni-stuttgart.de/mailman/listinfo/dumux

Reply via email to