thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-03 04:52:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix bugprone-macro-parentheses

Browse Source
This commit is contained in:
Sema Checherinda 2024-08-09 15:04:03 +02:00
parent 0939e7f922
commit 96b54df163
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Interpreters/SystemLog.cpp
View File

@ -284,11 +284,13 @@ ASTPtr getCreateTableQueryClean(const StorageID & table_id, ContextPtr context)
SystemLogs::SystemLogs(ContextPtr global_context, const Poco::Util::AbstractConfiguration & config) SystemLogs::SystemLogs(ContextPtr global_context, const Poco::Util::AbstractConfiguration & config)
{ {
/// NOLINTBEGIN(bugprone-macro-parentheses)
#define CREATE_PUBLIC_MEMBERS(log_type, member, descr) \ #define CREATE_PUBLIC_MEMBERS(log_type, member, descr) \
member = createSystemLog<log_type>(global_context, "system", #member, config, #member, descr); \ member = createSystemLog<log_type>(global_context, "system", #member, config, #member, descr); \
LIST_OF_ALL_SYSTEM_LOGS(CREATE_PUBLIC_MEMBERS) LIST_OF_ALL_SYSTEM_LOGS(CREATE_PUBLIC_MEMBERS)
#undef CREATE_PUBLIC_MEMBERS #undef CREATE_PUBLIC_MEMBERS
/// NOLINTEND(bugprone-macro-parentheses)
if (session_log) if (session_log)
global_context->addWarningMessage("Table system.session_log is enabled. It's unreliable and may contain garbage. Do not use it for any kind of security monitoring."); global_context->addWarningMessage("Table system.session_log is enabled. It's unreliable and may contain garbage. Do not use it for any kind of security monitoring.");
@ -333,7 +335,7 @@ SystemLogs::SystemLogs(ContextPtr global_context, const Poco::Util::AbstractConf
std::vector<ISystemLog *> SystemLogs::getAllLogs() const std::vector<ISystemLog *> SystemLogs::getAllLogs() const
{ {
#define GET_RAW_POINTERS(log_type, member, descr) \ #define GET_RAW_POINTERS(log_type, member, descr) \
member.get(), \ (member).get(), \
std::vector<ISystemLog *> result = { std::vector<ISystemLog *> result = {
LIST_OF_ALL_SYSTEM_LOGS(GET_RAW_POINTERS) LIST_OF_ALL_SYSTEM_LOGS(GET_RAW_POINTERS)

2
src/Interpreters/SystemLog.h
View File

@ -60,11 +60,13 @@ namespace DB
}; };
*/ */
/// NOLINTBEGIN(bugprone-macro-parentheses)
#define FORWARD_DECLARATION(log_type, member, descr) \ #define FORWARD_DECLARATION(log_type, member, descr) \
class log_type; \ class log_type; \
LIST_OF_ALL_SYSTEM_LOGS(FORWARD_DECLARATION) LIST_OF_ALL_SYSTEM_LOGS(FORWARD_DECLARATION)
#undef FORWARD_DECLARATION #undef FORWARD_DECLARATION
/// NOLINTEND(bugprone-macro-parentheses)
/// System logs should be destroyed in destructor of the last Context and before tables, /// System logs should be destroyed in destructor of the last Context and before tables,