D11326: [ToolTip] Check file name in KDirWatch handler

2018-03-14 Thread Kai Uwe Broulik
broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
broulik requested review of this revision.

REVISION SUMMARY
  `KDirWatch::self()` is a singleton so we would get change events also for 
other files like `plasma-appletsrc` needlessly reloading `plasmarc` all the 
time.

TEST PLAN
  It reloaded `plasmarc` 103 times on startup for me, now it only does it like 
3 times

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D11326

AFFECTED FILES
  src/declarativeimports/core/tooltip.cpp
  src/declarativeimports/core/tooltip.h

To: broulik, #plasma
Cc: #frameworks, michaelh, ngraham


D11326: [ToolTip] Check file name in KDirWatch handler

2018-03-14 Thread Kai Uwe Broulik
broulik edited the test plan for this revision.

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D11326

To: broulik, #plasma
Cc: #frameworks, michaelh, ngraham


D11326: [ToolTip] Check file name in KDirWatch handler

2018-03-15 Thread Marco Martin
mart accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D11326

To: broulik, #plasma, mart
Cc: #frameworks, michaelh, ngraham


D11326: [ToolTip] Check file name in KDirWatch handler

2018-03-15 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:77bac859bcec: [ToolTip] Check file name in KDirWatch 
handler (authored by broulik).

REPOSITORY
  R242 Plasma Framework (Library)

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D11326?vs=29492&id=29584

REVISION DETAIL
  https://phabricator.kde.org/D11326

AFFECTED FILES
  src/declarativeimports/core/tooltip.cpp
  src/declarativeimports/core/tooltip.h

To: broulik, #plasma, mart
Cc: #frameworks, michaelh, ngraham