Allow step definition in multiple source files about cucumber-cpp HOT 7 CLOSED

This is due to the fact that the compiler you are using does not support the __COUNTER__ macro. Take a look at include/cukebins/internal/RegistrationMacros.hpp:

#ifdef __COUNTER__
#define CUKE_GEN_OBJECT_NAME_ BOOST_JOIN(CukeObject, __COUNTER__)
#else
// Use a counter to be incremented every time cukebins is included
// in case this does not suffice (possible with multiple files only)
#define CUKE_GEN_OBJECT_NAME_ BOOST_JOIN(CukeObject, __LINE__)
#endif

from cucumber-cpp.

I use the gcc 4.2.1 from MacOS. It doesn't support the COUNTER macro.

It also doesn't work if you compile all source files serperately into .o files before linking.

Is it possible to include the file name into the object name?

from cucumber-cpp.

Legal characters for identifiers are letters, numbers, and _. The file name always includes at least one illegal character: the dot.

A workaround would be to include all your cpp files into a "global" one, but having to compile everything from scratch every time you change one file.

We might solve this issue by allowing the definition of a suffix in every file, by a define directive or a corresponding compiler parameter. A generated counter (followed by a legal character as the underscore) should not be difficult to add with cmake or scons.

from cucumber-cpp.

The suffix solution works. I did it exactly that way (but as prefix)*.
I added the following code to RegistrationMacros.hpp

#ifdef GEN_OBJECT_PREFIX
#define CUKE_GEN_OBJECT_NAME_ BOOST_JOIN(GEN_OBJECT_PREFIX,LINE)
#else

Then I used SCons to add the file name (without the suffix):

def buildObject(sourceFile):
return env.Object(
unicode('target/'+sourceFile.name[0:-4]+'.o'),
sourceFile,
CXXFLAGS='-DGEN_OBJECT_PREFIX=Object'+sourceFile.name[0:-4]
)

It would be great if you could add this solution to the official code base.

from cucumber-cpp.

Yes, that's exactly what I meant. I used CUKE_OBJECT_PREFIX instead of GEN_OBJECT_PREFIX because it's unlikely that anyone would want to name a macro in their code starting with CUKE :-) If not defined, it defaults to CukeObject as before.

from cucumber-cpp.

Great! Thanks

from cucumber-cpp.

can just add a anonymous namespace for each cpp file

from cucumber-cpp.