views:

368

answers:

5

Is it acceptable to add types to the std namespace. For example, I want a TCHAR-friendly string, so is the following acceptable?

#include <string>

namespace std
{
    typedef basic_string<TCHAR> tstring;
}

Or should I use my own namespace?

+3  A: 

You should use your own namespace as adding code to the standard library will only confuse the users that will look online for informations about that addition.

All that is in std should be only the standard library and nothing else.

Klaim
I'm not convinced it will confuse users - in fact they may expect a std::basic_string type to be in std. Tough call I think.
Rob
No, Klaim is correct. You should not add anything to the std namespace. Users may *expect* basic_string to be in std, but it is NOT part of the std library and they will not find anything on it in the published documentation for a given std implementation. The correct way is to use your own ns.
Burly
+11  A: 

No ... part of the point of a namespace is to prevent name collisions on upgrade.

If you add things to the std namespace, then your code might break with the next release of the library if they decide to add something with the same name.

Rob Walker
It also violates the standard, afaik. The std namespace is sacred. (Apart from specializations of existing std functions) :)
jalf
+2  A: 

Officially, the standard says that's "undefined behaviour", and all kinds of nasty things can happen.

In practice, it will work fine, but you still shouldn't do it. What does it buy you, other than confusing people that something is provided by the compiler?

Chris Jefferson
+2  A: 

Only specializations are allowed. So for example, you are allowed to specialize std::swap for your type. And this of course must happen in namespace std::. But your typedef isn't a specialization so that's causing undefined behavior.

Johannes Schaub - litb
A: 

I totally agree with other answers saying that you should put your types in your own namespace to avoid unfortunate name collisions.

However, I wanted to precise that sometimes, you can (and should !) add stuff in the std namespace. This is the case for template specializations of the std::swap method for example, which are used to provide a uniform way to swap objects. For more information on this matter, you can read about the non-throwing swap idiom.

Luc Touraille