I am using only header files specific to C++ (e.g. <cstdlib>
), however I still get globally-declared functions, and not just functions in the std
namespace. Is there a way, perhaps a compiler switch, to prevent that?
For example, the following code:
#include <cstdlib>
float random() { return 0.0f; }
int main() { return 0; }
Fails to compile under linux, with the following error:
> g++ -c main.cpp main.o
main.cpp: In function ‘float random()’:
main.cpp:2:14: error: new declaration ‘float random()’
/usr/include/stdlib.h:327:17: error: ambiguates old declaration ‘long int random()’
or
> clang++ main.cpp -o main.o
main.cpp:2:7: error: functions that differ only in their return type cannot be overloaded
float random() { return 0.0f; }
/usr/include/stdlib.h:327:17: note: previous declaration is here
extern long int random (void) __THROW;
which is caused that stdlib.h
"pollutes" the global namespace with its own random
function.
Note, that I am not facing these problems when compiling on Windows, using Visual Studio.
You can declare your functions in their own namespaces to prevent declaration collision.
namespace MyFunc
{
float random() { return 0.0f; }
};
In general you should try to avoid redeclaring in the first place.
You can do this by either using namespaces or by splitting up your source into files which can include cstdlib
and others which can use a static
version of your (name clashing) function.
If this is not an options then go on reading. But be aware that the following might be very platform specific.
By just having a look at my cstdlib
and stdlib.h
here at my place I noticed that there is a switch by which cstdlib
decides if it includes stdlib.h
or just declares abort
, atext
and exit
in the std
namespace.
Obviously you pull in the stdlib.h
branch. Looking further into this file I noticed the macro __BEGIN_NAMESPACE_STD
and later on __END_NAMESPACE_STD
. Maybe you could use this, but it is (as the name suggests) some implementation internal macro and should not be set directly by you. However, it should be there for some reason so you might have luck with searching for it.
After some more search it turned out that random
is one of several functions (and declarations) which are not wrapped into __BEGIN_NAMESPACE_STD
. Therefore, this is not a solution to the problem. (I found another macro _GLIBCPP_USE_NAMESPACES
which seems to be used internally as well to #define __BEGIN_NAMESPACE_STD namespace std {
).
So to sum it up: This is no viable path and you should use one of the described workarounds.
The standard explicitly permits <c???>
headers to bring names of C standard functions to the global namespace.
usually I would prefer to keep your function names different from what is defined as a standard .
For ex here one could use function name as myRandom instead of random so that I can inform the people , who would be maintaining my code later on , that the function being used is NOT the one defined as a standard.