Re: Notifications plasmoid, leak?

2010-06-06 Thread Marco Martin
On Sunday 06 June 2010, Alex Fiestas wrote: Hi fellows Seems that the notifications plasmoid is leaking X11 memory for each notification How to reproduce: 1-Execute system activity (Ctrl+Esc), and show the X11 memory column 2-Remove your notifications plasmoid inside plasma. 3-Execute

Re: Notifications plasmoid, leak?

2010-06-06 Thread Dave Lepore
I can reproduce it in trunk updated and compiled a couple of hours ago. Also, the issue is not reproducible in trunk compiled the 24th of may, so the cause should be a commit in this period of time. Hope this helps. Dave. On Sunday 06 June 2010 12:30:34 Marco Martin wrote: On Sunday 06 June

Review Request: use portable KDiskFreeSpaceInfo for free space calculation

2010-06-06 Thread Pino Toscano
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4241/ --- Review request for Plasma and Christopher Blauvelt. Summary --- Currently,

Re: Review Request: use portable KDiskFreeSpaceInfo for free space calculation

2010-06-06 Thread Pino Toscano
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4241/ --- (Updated 2010-06-06 13:15:14.345121) Review request for Plasma and Christopher

Re: Review Request: use portable KDiskFreeSpaceInfo for free space calculation

2010-06-06 Thread Alessandro Diaferia
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4241/#review6000 --- Ship it! Can't imagine anything saner than this. Of course last word

Re: [Plasma Media Center] some advice request

2010-06-06 Thread Martin Gräßlin
Am Freitag 04 Juni 2010, 23:47:07 schrieb Aaron J. Seigo: another approach might be to make it a kwin plugin, and have the video window rendered beneath the PMC window using a compositing plugin. this plugin would need to know what the video window is and what the PMC window is (can be done

Re: Review Request: use portable KDiskFreeSpaceInfo for free space calculation

2010-06-06 Thread Christopher Blauvelt
Seems fine to me. I think the reason we used unix specific libraries originally was we did not plan on supporting plasma on other OSs. Clearly that's no longer the case so this makes sense. On Sun, Jun 6, 2010 at 2:08 PM, Alessandro Diaferia alediafe...@gmail.comwrote: