conditional function signature definitions

[jmrubillon]

Hi all,

I have a header file with the following function declaration:

#ifdef DEBUG_ALLOC_MEMORY

void *AllocateMemoryInternal(uint32_t size, const char *file, int line);

#define AllocateMemory(size) \
    AllocateMemoryInternal(size,__FILE__,__LINE__)

#else

void *AllocateMemoryInternal(uint32_t size);
#defineAllocateMemory(size) AllocateMemoryInternal(size)

#endif

The same compilation flag switch is used in the corresponding .c file.

When I run ceedling (version 1.0.0) on this file with DEBUG_ALLOC_MEMORY not defined in the project.yml I get:

🧨 EXCEPTION: 'Default Test Compiler' (gcc) terminated with exit code [1] and output >>
build/test/mocks/test_service/mock_wrapper.c:8213:7: error: conflicting types for ‘AllocateMemoryInternal’; have ‘void *(uint32_t,  const char *, int)’ {aka ‘void *(unsigned int,  const char *, int)’}
 8213 | void* AllocateMemoryInternal(uint32_t size, const char* file, int line)
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from software/include/hal.h:20,
                 from build/test/mocks/test_service/mock_wrapper.h:12,
                 from build/test/mocks/test_service/mock_wrapper.c:6:
software/include/oswrapper.h:446:7: note: previous declaration of ‘AllocateMemoryInternal’ with type ‘void *(uint32_t)’ {aka ‘void *(unsigned int)’}
  446 | void *AllocateMemoryInternal(uint32_t size);
      |       ^~~~~~~~~~~~~~~~~~~~~~~~~~

But if I define DEBUG_ALLOC_MEMORY in my project.yml then the whole thing compiles just fine 🤔
No conflicting declarations warnings, nothing but a successful test run.

How can that be? And more importantly, how can I resolve this?

[mvandervoord]

Hi.

Out of curiosity, do you have :use_test_preprocessor set to :mocks or :all ? If not, it's not going to preprocess your header file before asking CMock to detect functions in it. It's very likely to find either or both versions in that case.

Otherwise I find it surprising that the problem is when it IS NOT defined, rather than when it IS defined... that makes it sound like you have a header or C file somewhere that is getting linked in that DOES define DEBUG_ALLOC_MEMORY ?

[jmrubillon]

Thanks for getting back to me.

I've tried a variety of settings and, providing I have DEBUG_ALLOC_MEMORY in my :defines: :test: section the code only compiles with :use_test_preprocessor as either :none or :tests. If I set it to :mocks or :all I get a raft of errors about macro and datatype redefinitions. Even when I put individual #ifndef, #define, #endif guards around them...

So, for now, I've settled on :use_test_preprocessor: :none and Defining DEBUG_ALLOC_MEMORY.

I've searched the entire code base for a #define DEBUG_ALLOC_MEMORY and it is nowhere to be found in either .c or .h files.