Review Request 129395: Add option to run an app on a discrete graphics card to KPropertiesDialog

[Jan Grulich]

> On Lis. 26, 2016, 7:22 odp., David Faure wrote:
> > src/widgets/kpropertiesdialog.cpp, line 3482
> > <https://git.reviewboard.kde.org/r/129395/diff/5/?file=486393#file486393line3482>
> >
> >     same comment as in 129399, it would be good if only the first properties dialog was slowed down by this dbus call, if the value can't change.

Isn't KPropertiesDialog initialized every time it's opened? Only solution that comes to my mind is to put this check outside, maybe use the value we already obtain in KRun? Or place it somewhere else where even KRun could read it?


- Jan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129395/#review101129
-----------------------------------------------------------


On Lis. 22, 2016, 12:06 odp., Jan Grulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129395/
> -----------------------------------------------------------
> 
> (Updated Lis. 22, 2016, 12:06 odp.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> This patch adds option to run certain application on a discrete graphics card (if there is any). I also modified KRun::runApplicationImpl() to take this option into account when running application.
> 
> 
> Diffs
> -----
> 
>   src/widgets/kpropertiesdesktopadvbase.ui 4d2a73d 
>   src/widgets/kpropertiesdialog.cpp 653383b 
> 
> Diff: https://git.reviewboard.kde.org/r/129395/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> Screenshot of KPropertiesDialog
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/14/89203f95-9267-47db-b2ad-0eeb42357a1c__kpropertiesdialog.png
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161128/e98d12b7/attachment.html>