diff options
| author | Katja Marttila <katja.marttila@qt.io> | 2021-03-05 08:58:04 +0200 |
|---|---|---|
| committer | Katja Marttila <katja.marttila@qt.io> | 2021-03-09 13:23:44 +0200 |
| commit | 6fdadf83710c37f70c1d2d621eacb9cbb327b23b (patch) | |
| tree | 3b0aede467044c529fae34e6c3f5ac654ea9f974 /tools/devtool/binarydump.cpp | |
| parent | abd97626ac2d50e54650a7256b48b7bbc1b2769e (diff) | |
Cppcheck: Fix local variable shadowing outer variable
Change-Id: Idff9a40c5089b4de7b8afd1c280603601317beda
Reviewed-by: Arttu Tarkiainen <arttu.tarkiainen@qt.io>
Diffstat (limited to 'tools/devtool/binarydump.cpp')
| -rw-r--r-- | tools/devtool/binarydump.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/devtool/binarydump.cpp b/tools/devtool/binarydump.cpp index bd8e9053c..bb0700149 100644 --- a/tools/devtool/binarydump.cpp +++ b/tools/devtool/binarydump.cpp @@ -117,10 +117,10 @@ int BinaryDump::dump(const QInstaller::ResourceCollectionManager &manager, const if ((!isOpen) && (!resource->open())) continue; // TODO: should we throw here? - QFile target(targetDir.filePath(name) + QDir::separator() + QFile targetFile(targetDir.filePath(name) + QDir::separator() + QString::fromUtf8(resource->name())); - QInstaller::openForWrite(&target); - resource->copyData(&target); // copy the 7z files into the target directory + QInstaller::openForWrite(&targetFile); + resource->copyData(&targetFile); // copy the 7z files into the target directory if (!isOpen) // If we reach that point, either the resource was opened already... resource->close(); // or we did open it and have to close it again. |