[Powerdevil] [Bug 383680] org_kde_powerdevil 45% ram usage along with 43% kded5 crashed my computer

2017-08-18 Thread iqbal
https://bugs.kde.org/show_bug.cgi?id=383680 --- Comment #1 from iqbal --- Created attachment 107357 --> https://bugs.kde.org/attachment.cgi?id=107357&action=edit htop report during last crash -- You are receiving this mail because: You are the assignee for the bug.

[Powerdevil] [Bug 383680] New: org_kde_powerdevil 45% ram usage along with 43% kded5 crashed my computer

2017-08-18 Thread iqbal
https://bugs.kde.org/show_bug.cgi?id=383680 Bug ID: 383680 Summary: org_kde_powerdevil 45% ram usage along with 43% kded5 crashed my computer Product: Powerdevil Version: unspecified Platform: Manjaro OS: Li

[Powerdevil] [Bug 351693] Screen brightness is not restored after screen unlock (idle time)

2017-08-18 Thread Yves Glodt
https://bugs.kde.org/show_bug.cgi?id=351693 Yves Glodt changed: What|Removed |Added CC||y...@mind.lu --- Comment #6 from Yves Glodt --- I

D7404: Run the Baloo runner out of process

2017-08-18 Thread David Edmundson
davidedmundson edited the summary of this revision. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D7404 To: davidedmundson, #plasma Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D7404: Run the Baloo runner out of process

2017-08-18 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY KRunner now supports querying data from running processes rather than being plugins. Due the number of crash reports of Baloo in

D7402: Use QUrls from match to create mimeData in base implementation

2017-08-18 Thread David Edmundson
davidedmundson created this revision. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel. REVISION SUMMARY QueryMatch contains a list of URLs. If an AbstractRunner doesn't implement this method, create some mimeData

D7398: Don't create QWhatsThis when user presses showContextHelp button

2017-08-18 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY I don't know what the idea of entering QWhatsThis mode was, but the code is pretty much dead. There is no QWhatsThis window shown as KWi

D7359: Send output enter/leave events to surfaces

2017-08-18 Thread Martin Flöser
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D7359 To: davidedmundson, #plasma, graesslin Cc: johanhelsing, graesslin, plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, ali-mohame

D7359: Send output enter/leave events to surfaces

2017-08-18 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D7359#136845, @davidedmundson wrote: > In https://phabricator.kde.org/D7359#136559, @graesslin wrote: > > > I'm wondering whether we should do it directly in Toplevel as we also have the XWayland windows which have a Surface.

D7359: Send output enter/leave events to surfaces

2017-08-18 Thread Martin Flöser
graesslin added a comment. In https://phabricator.kde.org/D7359#136850, @davidedmundson wrote: > One question that I'm unsure of: subsurfaces. > > A subsurface is still a surface, so from reading the API arguably they should also get enter/leave events...but at the same time it seems

D7369: [WIP] Wayland foreign protocol

2017-08-18 Thread Marco Martin
mart added a comment. part of the comments adressed, still to wrap the client part in order to hide the v1 implementation. the status right now is that, when running in valgrind all test pass, while then invoked directly it almost always crashes, but every time at a different, random pl

D7369: [WIP] Wayland foreign protocol

2017-08-18 Thread Marco Martin
mart updated this revision to Diff 18357. mart added a comment. Restricted Application edited projects, added Plasma; removed Plasma on Wayland. - get rid of the v1 from the verver side - dress some issues - Q_DECL_HIDDEN for all non exported classes - track surface destruction - more a

D7395: [tabBox] Move X11 specific event filtering into a dedicated event filter

2017-08-18 Thread Martin Flöser
graesslin added a dependency: D7374: Move X11 movingClient handling into a dedicated X11EventFilter. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D7395 To: graesslin, #kwin, #plasma Cc: plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreut

D7374: Move X11 movingClient handling into a dedicated X11EventFilter

2017-08-18 Thread Martin Flöser
graesslin added a dependent revision: D7395: [tabBox] Move X11 specific event filtering into a dedicated event filter. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D7374 To: graesslin, #kwin, #plasma Cc: mart, broulik, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai,

D7395: [tabBox] Move X11 specific event filtering into a dedicated event filter

2017-08-18 Thread Martin Flöser
graesslin created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: kwin, plasma-devel. REVISION SUMMARY This ensures that the X11 specific events are only read on X11 but not on Wayland. Also moves quite some X11 specific code out of TabBox

[kio-extras] [Bug 372860] MTP not usable : connects, then disconnects, then connects...

2017-08-18 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=372860 Nate Graham changed: What|Removed |Added CC||cbm4...@gmail.com --- Comment #2 from Nate Graham

D7377: Track outputs in kwin integration tests

2017-08-18 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R108:34de2c2b5c8b: Track outputs in kwin integration tests (authored by davidedmundson). REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7377?vs=18311&id=18345 REVISION DET

D7379: KWayland::Client::Surface track output destruction

2017-08-18 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R127:e430a44c4a92: KWayland::Client::Surface track output destruction (authored by davidedmundson). REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7379?vs=18315&id=1834

D6134: Test every plasmoid

2017-08-18 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R527:31c85ca49872: Test every plasmoid (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D6134?vs=15247&id=18341#toc REPOSITORY R527 Integration tests for Plasma CHANGE

D7374: Move X11 movingClient handling into a dedicated X11EventFilter

2017-08-18 Thread Marco Martin
mart added inline comments. INLINE COMMENTS > graesslin wrote in moving_client_x11_filter.cpp:36 > *shrug* honestly I don't see that the one or the other is better. The old > code used dynamic_cast and I try to be as close as possible to the old one. both are fine, qobject_cast should be a tad

D7379: KWayland::Client::Surface track output destruction

2017-08-18 Thread Martin Flöser
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D7379 To: davidedmundson, #plasma, graesslin Cc: plasma-devel, #frameworks, leezu, ZrenBot, progwolff, lesliezhai, ali-moh

D7377: Track outputs in kwin integration tests

2017-08-18 Thread Martin Flöser
graesslin accepted this revision. This revision is now accepted and ready to land. REPOSITORY R108 KWin BRANCH master REVISION DETAIL https://phabricator.kde.org/D7377 To: davidedmundson, #plasma, graesslin Cc: plasma-devel, kwin, #kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, harde

D7374: Move X11 movingClient handling into a dedicated X11EventFilter

2017-08-18 Thread Martin Flöser
graesslin added inline comments. INLINE COMMENTS > broulik wrote in moving_client_x11_filter.cpp:36 > `qobject_cast` *shrug* honestly I don't see that the one or the other is better. The old code used dynamic_cast and I try to be as close as possible to the old one. REPOSITORY R108 KWin REV

D7385: Compress size hint changes before releasing position and repositioning item.

2017-08-18 Thread Eike Hein
This revision was automatically updated to reflect the committed changes. Closed by commit R119:3e3f0decc983: Compress size hint changes before releasing position and repositioning item. (authored by hein). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/

D7384: Update size hints less often.

2017-08-18 Thread Eike Hein
This revision was automatically updated to reflect the committed changes. Closed by commit R119:b8c1e0e3e060: Update size hints less often. (authored by hein). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7384?vs=18333&id=18335 REVISION DETAIL http

D7384: Update size hints less often.

2017-08-18 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop BRANCH Plasma/5.10 REVISION DETAIL https://phabricator.kde.org/D7384 To: hein, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D7385: Compress size hint changes before releasing position and repositioning item.

2017-08-18 Thread Marco Martin
mart accepted this revision. mart added a comment. This revision is now accepted and ready to land. a timer always makes things more complicated, but.. those sizes should really be updated all at once and since qml doesn't have a way to do it, ok for a timer REPOSITORY R119 Plasma Desktop

D7385: Compress size hint changes before releasing position and repositioning item.

2017-08-18 Thread Eike Hein
hein added inline comments. INLINE COMMENTS > AppletAppearance.qml:164 > root.layoutManager.positionItem(appletItem); > -root.layoutManager.saveItem(appletItem); > } This line was unnecessary, layoutManager.positionItem saves at the end. REPOSITORY R119 Plasma Desktop R

D7385: Compress size hint changes before releasing position and repositioning item.

2017-08-18 Thread Eike Hein
hein created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY https://phabricator.kde.org/D4204 / https://phabricator.kde.org/R119:d7621da69b9b4758b3e877f78545645bcb1c4305 introduced more elaborate handling of item size hint changes, releasing and repositioning

D7384: Update size hints less often.

2017-08-18 Thread Eike Hein
hein created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY The size hints were based on the size of pagerGridItem, which is derived from its contents. This made the size hints update as delegates were added, but we know ahead of time how many there will be,

D7304: Do not leak symbols of pimpl classes, protect with Q_DECL_HIDDEN

2017-08-18 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R127:b7a63491ad1c: Do not leak symbols of pimpl classes, protect with Q_DECL_HIDDEN (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D7304?vs=18133&id=18330#toc REPOSITOR

D7369: [WIP] Wayland foreign protocol

2017-08-18 Thread Marco Martin
mart marked 2 inline comments as done. mart added inline comments. INLINE COMMENTS > davidedmundson wrote in xdgforeign_v1_interface.cpp:299 > this should be connected to unbound. perhaps the connects to the destruction of the SurfaceInterface* are not even necessary? REPOSITORY R127 KWaylan

D7369: [WIP] Wayland foreign protocol

2017-08-18 Thread Marco Martin
mart added inline comments. INLINE COMMENTS > graesslin wrote in registry.h:938-939 > The general pattern I used for protocols which might get another revision is > to create a "meta" class which hides the UnstableV1 mess. C.f. the > PointerConstraints a line above. i see the types are named z

D7361: Get rid of QSignalMapper

2017-08-18 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R242:1419135a5d99: Get rid of QSignalMapper (authored by broulik). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7361?vs=18280&id=18327 REVISION DET

D7381: Workaround bug in QT_QUICK_CONTROLS_1_STYLE usage in Qt

2017-08-18 Thread Fabian Vogt
fvogt closed this revision. fvogt added a comment. Even `arc land` doesn't close diffs reliably now apparently... REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D7381 To: fvogt, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jen

D7381: Workaround bug in QT_QUICK_CONTROLS_1_STYLE usage in Qt

2017-08-18 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D7381 To: fvogt, #plasma, mart Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, s

D7381: Workaround bug in QT_QUICK_CONTROLS_1_STYLE usage in Qt

2017-08-18 Thread Fabian Vogt
fvogt created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY Setting the style manually is necessary as there is only a variable that applies to both quick controls versions, so it needs to be overwritten to not cause a warning on every application start. The