search

meekrosoft / fff

A testing micro framework for creating function test doubles
Other
768 stars 169 forks source link

Remove consts from defined structs to avoid errors with c++ #85

Open sebirdman opened 4 years ago

sebirdman commented 4 years ago

This should properly resolve https://github.com/meekrosoft/fff/issues/51

AppVeyorBot commented 4 years ago

:x: Build fff 85-appveyor failed (commit https://github.com/meekrosoft/fff/commit/b27f8d19ad by @sebirdman)

AppVeyorBot commented 4 years ago

:x: Build fff 86-appveyor failed (commit https://github.com/meekrosoft/fff/commit/6fb172869f by @sebirdman)

sebirdman commented 4 years ago

I'm not sure what the tests are doing exactly, i'm not able to reproduce these failures on my machine

johwech commented 4 years ago

The changes introduced by the original pull request require the fff.h to be not included within an extern "C" block when compiling with a cpp compiler.

I fixed the issue by wrapping an additional extern "C++" block around the whole fff.h: https://ci.appveyor.com/project/johwech/fff/builds/32597466

Alternatively it's also possible to just fix the test and tell the users that fff.h cannot be included within an extern "C" segment when a CPP compiler is used: https://ci.appveyor.com/project/johwech/fff/builds/32597324

gndl commented 2 years ago

Hi, first of all thank you for this great tool. This PR allows me to use FFF, how can I help to get it integrated mainstream?