cactusdynamics / cactus-rt

A C++ framework for programming real-time applications
Mozilla Public License 2.0
185 stars 24 forks source link

fix duplicate header guards #146

Open Nibanovic opened 1 month ago

Nibanovic commented 1 month ago

sink.h and sink.disabled.h have identical header guards, because of which I couldn't build my project with cactus-rt linked.

This was the only issue and the build then succeeded as expected.

As the CACTUS_RT_TRACING_ENABLED compiler option is set by default, these header should not be included. I'm not sure why the compiler then went ahead and looked for required definitions in ALL headers, even those that should not be included - I'm a bit out of my depth here.

Anyways, now it works for me and is synchronized with how the rest of the header guards in include/tracing are defined.

Byeee