iox-#1755 Reintroduce lazy evaluation for the logging

[elBoberido]

Pre-Review Checklist for the PR Author

1. Code follows the coding style of CONTRIBUTING.md
2. Tests follow the best practice for testing
3. Changelog updated in the unreleased section including API breaking changes
4. Branch follows the naming format (iox-123-this-is-a-branch)
5. Commits messages are according to this guideline
   • Commit messages have the issue ID (iox-#123 commit text)
   • Commit author matches Eclipse Contributor Agreement (and ECA is signed)
6. Update the PR title
   • Follow the same conventions as for commit messages
   • Link to the relevant issue
7. Relevant issues are linked
8. Add sensible notes for the reviewer
9. All checks have passed (except task-list-completed)
10. All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
11. Assign PR to reviewer

Notes for Reviewer

This PR introduces a new macro for logging which utilizes lazy evaluation. The port to the new macro will happen in a follow up PR.

Checklist for the PR Reviewer

• Commits are properly organized and messages are according to the guideline
• Code according to our coding style and naming conventions
• Unit tests have been written for new behavior
• Public API changes are documented via doxygen
• Copyright owner are updated in the changed files
• All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
• PR title describes the changes

Post-review Checklist for the PR Author

1. All open points are addressed and tracked via issues

References

• Relates to Enhance Logger #1755

[elBoberido]

@FerdinandSpitzschnueffler @mossmaurice @dkroenke @MatthiasKillat @elfenpiff feel free to add yourself as reviewer

[codecov]

Codecov Report

Merging #2037 (2a5da9a) into master (4fb0b5f) will decrease coverage by 1.52%.
Report is 9 commits behind head on master.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2037      +/-   ##
==========================================
- Coverage   74.23%   72.71%   -1.52%     
==========================================
  Files         417      417              
  Lines       16290    16285       -5     
  Branches     2277     2887     +610     
==========================================
- Hits        12093    11842     -251     
- Misses       3458     3469      +11     
- Partials      739      974     +235     

Flag	Coverage Δ
unittests	72.50% <0.00%> (-1.52%)	⬇️
unittests_timing	15.03% <0.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...ofs/reporting/include/iox/detail/log/logstream.inl	95.68% <ø> (-0.11%)	⬇️
iceoryx_hoofs/reporting/include/iox/logging.hpp	50.00% <ø> (ø)
...error_reporting/custom/default/error_reporting.inl	73.21% <0.00%> (-14.72%)	⬇️

... and 69 files with indirect coverage changes

[FerdinandSpitzschnueffler]

LGTM, only nitpicks.

[elBoberido]

@mossmaurice @dkroenke @MatthiasKillat @elfenpiff anyone who would like to do the second review?

We desperately need to change the number of reviewers. There is another open PR which is based on this one and I have a further branch which is ready to review. With this speed I won't be able to get this in this year and I'm currently blocked by the slow review process.

[elBoberido]

@mossmaurice @dkroenke @MatthiasKillat @elfenpiff anyone?

[mossmaurice]

LGTM, thanks!