dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> kidletime.cpp:253
>  
>      if (associations.values().contains(msec)) {
> +        const auto listKeys = associations.keys(msec);

Urgh, how horrible and slow. But pre-existing, unrelated to your change.

REPOSITORY
  R274 KIdleTime

BRANCH
  compile_without_foreach (branched from master)

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

To: mlaurent, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to