D10038: Use QTemporaryDir::filePath() instead of path concatenation
volkov closed this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10038 To: volkov, #plasma, hein Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10038: Use QTemporaryDir::filePath() instead of path concatenation
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10038 To: volkov, #plasma, hein Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10038: Use QTemporaryDir::filePath() instead of path concatenation
volkov added a comment. ping REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D10038 To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
D10038: Use QTemporaryDir::filePath() instead of path concatenation
volkov created this revision. volkov added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. volkov requested review of this revision. REVISION SUMMARY This is the preferred way to get a file path in a temporary directory. It's easy to forget to append '/' when using QTemporaryDir::path(). REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.12 REVISION DETAIL https://phabricator.kde.org/D10038 AFFECTED FILES kcms/kfontinst/kcmfontinst/FontsPackage.cpp kcms/kfontinst/kcmfontinst/JobRunner.cpp kcms/kfontinst/kio/KioFonts.cpp kcms/kfontinst/thumbnail/FontThumbnail.cpp kcms/kfontinst/viewpart/FontViewPart.cpp To: volkov, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart