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

David Faure faure at kde.org
Mon Nov 28 07:37:46 UTC 2016



> On Nov. 26, 2016, 7:22 p.m., 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.
> 
> Jan Grulich wrote:
>     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?

"Isn't KPropertiesDialog initialized every time it's opened?"
=> it is, but that doesn't matter. The whole point of a static variable is that it survives all instances of the class.


- David


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


On Nov. 22, 2016, 12:06 p.m., Jan Grulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129395/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2016, 12:06 p.m.)
> 
> 
> 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/1ac7cd73/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list