Skip to content

Commit

Permalink
Fixed build for gcc
Browse files Browse the repository at this point in the history
`error: control reaches end of non-void function`
  • Loading branch information
Enmk authored Feb 17, 2022
1 parent 9c3c33e commit 8e3970c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions clickhouse/types/types.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,9 @@ uint64_t Type::GetTypeUniqueId() const {
case LowCardinality: {
// For complex types, exact unique ID depends on nested types and/or parameters,
// the easiest way is to lazy-compute unique ID from name once.
// Here we do not care if multiple thread are computing value simultaneosly since:
// 1. it is going to be the same
// 2. it is going to be stored atomically
// Here we do not care if multiple threads are computing value simultaneosly since it is both:
// 1. going to be the same
// 2. going to be stored atomically

if (type_unique_id_ == 0) {
const auto name = GetName();
Expand All @@ -129,6 +129,8 @@ uint64_t Type::GetTypeUniqueId() const {
return type_unique_id_;
}
}
assert(false);
return 0;
}

TypeRef Type::CreateArray(TypeRef item_type) {
Expand Down

0 comments on commit 8e3970c

Please sign in to comment.