amantia added inline comments.

INLINE COMMENTS

> mwolff wrote in screenmapper.cpp:41
> this is a pretty arbitrary timer, can you add a comment on why 100ms is 
> better than going through the eventloop once via QMetaObject::invokeMethod 
> with the delayed flag set?

Done. InvokeMethod would not help, as the idea here is to get only one 
screenMappingChanged instead of e.g 50 when FolderModel gets the entries on 
startup.

REPOSITORY
  R119 Plasma Desktop

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

To: amantia, #plasma, ervin, mlaurent, dvratil, hein, aacid, davidedmundson, 
apol, mwolff
Cc: anthonyfieroni, ngraham, mwolff, davidedmundson, broulik, mart, 
plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol

Reply via email to