Release configuration build fails

ekcoh / gtest-memleak-detector

Google Test extension providing memory leak detection and debugging using MSVC CRT debug.

MIT License

3 stars 0 forks source link

Release configuration build fails #2

Open ekcoh opened 4 years ago

ekcoh commented 4 years ago

Build fails for Release configurations. This should be easy to remedy since compiler is complaining about unused arguments (trivial to fix with preprocessor condition) and some unfortunate incorrect naming in the #else branches which is also trivial.

As part of this task both Debug and Release builds should of course be part of CI which was overlooked in initial commits.

ekcoh commented 4 years ago

Partially solved on master. CI part remains unresolved.

ekcoh commented 3 years ago

Fixed a related issue https://github.com/ekcoh/gtest-memleak-detector/issues/5 which indicated build problems for Release builds that was previously undetected since only RelWithDebInfo was considered. This was overseen since the library only is useful in Debug modes, but should not cause problems for any Release target build.