diff options
| author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2024-07-05 10:35:57 +0200 |
|---|---|---|
| committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2024-07-06 13:10:36 +0200 |
| commit | 6346c5b426935a767575ea758e99749bc6544f54 (patch) | |
| tree | e222b7d89915fc2b451f8fe5a6b4b7f9c02d839d /src/tools/uic/python/pythonwriteimports.cpp | |
| parent | d1a5e602df5978cee6bd263db77745f8047e508e (diff) | |
uic: Fix some clang-tidy warnings
- Repeated return types
- Make member functions static
- narrowing int conversions
- Use auto *
- Use list.isEmpty()
- Use modern includes
- Remove redundant access specifiers
- minor fixes
Pick-to: 6.8
Change-Id: I14ddf1add667536739fbb5fabb357dbaa7ef35c3
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
Diffstat (limited to 'src/tools/uic/python/pythonwriteimports.cpp')
| -rw-r--r-- | src/tools/uic/python/pythonwriteimports.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/tools/uic/python/pythonwriteimports.cpp b/src/tools/uic/python/pythonwriteimports.cpp index 74eeab8387e..3c6f5710cee 100644 --- a/src/tools/uic/python/pythonwriteimports.cpp +++ b/src/tools/uic/python/pythonwriteimports.cpp @@ -124,9 +124,9 @@ void WriteImports::acceptUI(DomUI *node) output << "import " << w << '\n'; } - if (auto resources = node->elementResources()) { - const auto includes = resources->elementInclude(); - for (auto include : includes) { + if (auto *resources = node->elementResources()) { + const auto &includes = resources->elementInclude(); + for (auto *include : includes) { if (include->hasAttributeLocation()) writeResourceImport(include->attributeLocation()); } |