diff options
| author | Ed Tanous <ed@tanous.net> | 2024-04-07 18:38:44 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2024-04-09 01:46:24 +0300 |
| commit | 6ea9076048028a4adfdbdd2606698bffc81203d3 (patch) | |
| tree | 23ba9504c963534c9354e66cdd61b19a5608eafd /.clang-tidy | |
| parent | 8cb2c024c4625e2fe2f0b107a865faffcd4bb770 (diff) | |
| download | bmcweb-6ea9076048028a4adfdbdd2606698bffc81203d3.tar.xz | |
Rework logger to create compile time errors
The logger changes to move to std::format incidentally caused format
errors to no longer be flagged at compile time.
The example here[1] shows that the latest gcc/c++ gave us a way to solve
this, using std::format_string.
This incidentally shows two places where we got the format arguments
wrong, so fix them.
[1] https://stackoverflow.com/questions/72795189/how-can-i-wrap-stdformat-with-my-own-template-function
Change-Id: Id884200e2c98eeaf5ef8db6c1d6362ede2ffb858
Signed-off-by: Ed Tanous <ed@tanous.net>
Diffstat (limited to '.clang-tidy')
| -rw-r--r-- | .clang-tidy | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/.clang-tidy b/.clang-tidy index 5b530c3791..0027ca00b2 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -355,6 +355,7 @@ CheckOptions: - { key: readability-identifier-naming.NamespaceCase, value: lower_case } - { key: readability-identifier-naming.StructCase, value: CamelCase } - { key: readability-identifier-naming.FunctionIgnoredRegexp, value: (BMCWEB_LOG_DEBUG|BMCWEB_LOG_INFO|BMCWEB_LOG_WARNING|BMCWEB_LOG_ERROR|BMCWEB_LOG_CRITICAL) } + - { key: readability-identifier-naming.StructIgnoredRegexp, value: (BMCWEB_LOG_DEBUG|BMCWEB_LOG_INFO|BMCWEB_LOG_WARNING|BMCWEB_LOG_ERROR|BMCWEB_LOG_CRITICAL) } - { key: cppcoreguidelines-macro-usage.AllowedRegexp, value: DEBUG*|NLOHMANN_JSON_SERIALIZE_ENUM } - { key: performance-unnecessary-value-param.AllowedTypes, value: ((segments_view)|(url_view)) } - { key: cppcoreguidelines-rvalue-reference-param-not-moved.IgnoreUnnamedParams, value: true } |