Fix some clang-tidy warnings in shiboken

- 'else if' after return/break/continue
- Use const ref for complex arguments passed by value where possible
- Fix 'if (foo) delete foo' to 'delete foo'
- Use container.isEmpty() instead of container.size() in checks
- Use ' = default' for trivial constructors/destructors
- Use range based for where possible
- Complete constructor initializer lists
- Fix invocations of static methods
- Replace some reinterpret_cast by static_cast
- Remove unused variables/fields
- Use initializer lists for return types

Change-Id: Id5b44a2f9d429f66ef069d532a1cd31df796d38e
Reviewed-by: Christian Tismer <tismer@stackless.com>

1 files changed, 0 insertions, 1 deletions

diff --git a/sources/shiboken2/generator/main.cpp b/sources/shiboken2/generator/main.cpp
index 67d838ce8..c996c7cc3 100644
--- a/sources/shiboken2/generator/main.cpp
+++ b/sources/shiboken2/generator/main.cpp
@@ -386,7 +386,6 @@ static void parseIncludePathOption(const QString &option, HeaderType headerType,
 {
     const CommandArgumentMap::iterator it = args.find(option);
     if (it != args.end()) {
-        const QString path = it.value();
         const QStringList includePathListList =
             it.value().split(QLatin1String(PATH_SPLITTER), QString::SkipEmptyParts);
         args.erase(it);