path: root/src/corelib/tools/qflatmap_p.h
diff options
authorLars Knoll <>2020-11-09 17:27:04 +0100
committerLars Knoll <>2020-11-17 11:46:28 +0100
commit621c05e3b1869a524c1617fe9d1d178af3f7f227 (patch)
tree0fbfa0470e670a4550db2c6ea8c2a26885dbd6a9 /src/corelib/tools/qflatmap_p.h
parent872f18bbd66d32862b9c10805ee125c466671df4 (diff)
Don't allow storing types that throw in the destructor in our containers
Types that throw in their destructors are strongly discouraged in C++, and even the STL doesn't define what happens if such types are stored in their containers. Make this more explicit for Qt and disallow storing those types in our containers. This will hopefully preempty any potential future bug reports about us not handling such a case. It also helps simplify some code in QList and other cases and makes it possible to explicitly mark more methods as noexcept. Some care needs to be taken where to add the static asserts, so that we don't disallow forward declarations of types stored in containers. Place the static assert into the destructor of the container where possible or otherwise into the templated d-pointer. Change-Id: If3aa40888f668d0f1b6c6b3ad4862b169d31280e Reviewed-by: Thiago Macieira <> Reviewed-by: Andrei Golubev <> Reviewed-by: MÃ¥rten Nordheim <>
Diffstat (limited to 'src/corelib/tools/qflatmap_p.h')
1 files changed, 2 insertions, 0 deletions
diff --git a/src/corelib/tools/qflatmap_p.h b/src/corelib/tools/qflatmap_p.h
index bf0efb2543..1b3eaea01c 100644
--- a/src/corelib/tools/qflatmap_p.h
+++ b/src/corelib/tools/qflatmap_p.h
@@ -109,6 +109,8 @@ template<class Key, class T, class Compare = std::less<Key>, class KeyContainer
class MappedContainer = QList<T>>
class QFlatMap : private QFlatMapValueCompare<Key, T, Compare>
+ static_assert(std::is_nothrow_destructible_v<T>, "Types with throwing destructors are not supported in Qt containers.");
using full_map_t = QFlatMap<Key, T, Compare, KeyContainer, MappedContainer>;
template <class U>