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