diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-08-30 16:48:38 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-08-30 16:36:08 +0000 |
commit | 3b6e2a30e5f0174a963c36c85dc8867f1322d345 (patch) | |
tree | 06c7d80187c6ce20edc2dce8663b1b92443137f5 /sources/shiboken2/libshiboken/bindingmanager.cpp | |
parent | 7e2200bf14e4dfc4fc65be154931e990912ca976 (diff) |
libshiboken: Use empty() to check for empty containers as clang-tidy advises
Using the size() method for that purpose can be expensive for standard
containers.
Task-number: PYSIDE-727
Change-Id: I0da34e271503384a741d856fff5e84fee67bc97f
Reviewed-by: Christian Tismer <tismer@stackless.com>
Diffstat (limited to 'sources/shiboken2/libshiboken/bindingmanager.cpp')
-rw-r--r-- | sources/shiboken2/libshiboken/bindingmanager.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sources/shiboken2/libshiboken/bindingmanager.cpp b/sources/shiboken2/libshiboken/bindingmanager.cpp index 2cc7847a8..0aa520b38 100644 --- a/sources/shiboken2/libshiboken/bindingmanager.cpp +++ b/sources/shiboken2/libshiboken/bindingmanager.cpp @@ -197,7 +197,7 @@ BindingManager::~BindingManager() while (!m_d->wrapperMapper.empty()) { Object::destroy(m_d->wrapperMapper.begin()->second, const_cast<void*>(m_d->wrapperMapper.begin()->first)); } - assert(m_d->wrapperMapper.size() == 0); + assert(m_d->wrapperMapper.empty()); } delete m_d; } |