Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EvilNumber Arithmetic functions EvilNumber::pow should be stripped of its EvilNumber namespace. #46

Closed
Captnoord opened this issue Jan 17, 2013 · 2 comments
Labels

Comments

@Captnoord
Copy link
Collaborator

When we can do
result = pow(EvilNumber(39.0), EvilNumber(2.0));

instead of
result = EvilNumber::pow(EvilNumber(39.0), EvilNumber(2.0));

it would be easier to read.

@aknorjaden
Copy link
Collaborator

Captnoord,

We've chosen to not make use of any "using namespace XXX;" anywhere in the code as a sort of coding standard, that way any functions belonging to a namespace are explicitly known. This is decent in large projects, or so we think. This is why you'll see a lot of std::map<uint32, uint32> or boost::asio::whatever.

Aknor

@aknorjaden
Copy link
Collaborator

Well, with no challenge to that, we're going to stick with explicitly calling out namespaces to avoid ambiguity of what the code is doing.

Closing this issue as Will not fix.

-- Aknor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants