mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-10 01:25:21 +00:00
Added some warnings from clang's -Weverything
This commit is contained in:
parent
67c236b067
commit
afda68d25c
@ -328,7 +328,7 @@ bool FunctionArrayEnumerateExtended<Derived>::execute128bit(
|
|||||||
size_t rank = 0;
|
size_t rank = 0;
|
||||||
for (size_t j = prev_off; j < off; ++j)
|
for (size_t j = prev_off; j < off; ++j)
|
||||||
{
|
{
|
||||||
auto &idx = indices[packFixed<UInt128>(j, count, columns, key_sizes)];;
|
auto & idx = indices[packFixed<UInt128>(j, count, columns, key_sizes)];
|
||||||
if (!idx)
|
if (!idx)
|
||||||
idx = ++rank;
|
idx = ++rank;
|
||||||
res_values[j] = idx;
|
res_values[j] = idx;
|
||||||
|
@ -268,7 +268,7 @@ void AsynchronousMetrics::update()
|
|||||||
set("jemalloc." NAME, value); \
|
set("jemalloc." NAME, value); \
|
||||||
} while (0);
|
} while (0);
|
||||||
|
|
||||||
FOR_EACH_METRIC(GET_METRIC);
|
FOR_EACH_METRIC(GET_METRIC)
|
||||||
|
|
||||||
#undef GET_METRIC
|
#undef GET_METRIC
|
||||||
#undef FOR_EACH_METRIC
|
#undef FOR_EACH_METRIC
|
||||||
|
@ -43,7 +43,7 @@ static const char * cancellationCodeToStatus(CancellationCode code)
|
|||||||
return "waiting";
|
return "waiting";
|
||||||
default:
|
default:
|
||||||
return "unknown_status";
|
return "unknown_status";
|
||||||
};
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@ -157,7 +157,7 @@ void StorageDistributedDirectoryMonitor::run()
|
|||||||
std::chrono::milliseconds{Int64(default_sleep_time.count() * std::exp2(error_count))},
|
std::chrono::milliseconds{Int64(default_sleep_time.count() * std::exp2(error_count))},
|
||||||
std::chrono::milliseconds{max_sleep_time});
|
std::chrono::milliseconds{max_sleep_time});
|
||||||
tryLogCurrentException(getLoggerName().data());
|
tryLogCurrentException(getLoggerName().data());
|
||||||
};
|
}
|
||||||
|
|
||||||
if (do_sleep)
|
if (do_sleep)
|
||||||
cond.wait_for(lock, sleep_time, quit_requested);
|
cond.wait_for(lock, sleep_time, quit_requested);
|
||||||
|
@ -68,7 +68,7 @@ void MergeTreeSettings::loadFromQuery(ASTStorage & storage_def)
|
|||||||
== changes.end()) \
|
== changes.end()) \
|
||||||
changes.push_back(ASTSetQuery::Change{#NAME, NAME.value});
|
changes.push_back(ASTSetQuery::Change{#NAME, NAME.value});
|
||||||
|
|
||||||
APPLY_FOR_IMMUTABLE_MERGE_TREE_SETTINGS(ADD_IF_ABSENT);
|
APPLY_FOR_IMMUTABLE_MERGE_TREE_SETTINGS(ADD_IF_ABSENT)
|
||||||
#undef ADD_IF_ABSENT
|
#undef ADD_IF_ABSENT
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -69,7 +69,7 @@ public:
|
|||||||
if (unlikely(isNull()))
|
if (unlikely(isNull()))
|
||||||
throwException("Value is NULL");
|
throwException("Value is NULL");
|
||||||
|
|
||||||
return readUIntText(m_data, m_length);;
|
return readUIntText(m_data, m_length);
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Получить целое со знаком или дату или дату-время (в unix timestamp согласно текущей тайм-зоне).
|
/// Получить целое со знаком или дату или дату-время (в unix timestamp согласно текущей тайм-зоне).
|
||||||
@ -190,7 +190,7 @@ private:
|
|||||||
|
|
||||||
Int64 getIntImpl() const
|
Int64 getIntImpl() const
|
||||||
{
|
{
|
||||||
return readIntText(m_data, m_length);;
|
return readIntText(m_data, m_length);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user