sofa icon indicating copy to clipboard operation
sofa copied to clipboard

Removing comments in the end of namespace...

Open damienmarchal opened this issue 2 years ago • 3 comments

Fell free to express you opinion.

damienmarchal avatar Sep 14 '22 09:09 damienmarchal

Basically, do you prefer

namespace sofa::foo::bar
{
...
} // namespace sofa::foo::bar

or

namespace sofa::foo::bar
{
...
}

?

alxbilger avatar Oct 05 '22 14:10 alxbilger

+1 for the last one :)

damienmarchal avatar Oct 05 '22 14:10 damienmarchal

So as in real life... if no one vote for... only the one that voted win the election.

damienmarchal avatar Oct 13 '22 11:10 damienmarchal

from what I remember, the } // namespace ... were useful for scripts to rename/remove some namespace level. But as now we can use (I don't know the exact name) multilevel namespace sofa::component:myawsomecomponentnamespace { this makes less sens.

The only use I see is for quick readability But why is it annoying you?

epernod avatar Nov 25 '22 08:11 epernod