D28778: [FstabWatcher] Use static QStringLiteral instead of macro
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Sun Apr 12 19:17:56 BST 2020
bruns created this revision.
bruns added a reviewer: Frameworks.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
bruns requested review of this revision.
REVISION SUMMARY
QFilesystemWatcher::addPath takes a QString as argument, so avoid
construction of tempary QString on each call.
This also avoids a name clash between the Solid::Backends::Fstab::FSTAB
logging category and the FSTAB macro.
TEST PLAN
no functional changes
REPOSITORY
R245 Solid
BRANCH
submit
REVISION DETAIL
https://phabricator.kde.org/D28778
AFFECTED FILES
src/solid/devices/backends/fstab/fstabwatcher.cpp
To: bruns, #frameworks
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200412/6a374bfc/attachment.html>
More information about the Kde-frameworks-devel
mailing list