Review Request 118812: [plasmashell] Show a warning if there are no Shaders and exit

David Edmundson david at davidedmundson.co.uk
Wed Jun 18 13:12:22 UTC 2014



> On June 18, 2014, 1 p.m., David Edmundson wrote:
> > shell/shellcorona.h, line 87
> > <https://git.reviewboard.kde.org/r/118812/diff/1/?file=282338#file282338line87>
> >
> >     private?
> 
> Martin Gräßlin wrote:
>     are private slots invokable through QMetaObject::invokeMethod?

sure.
>From the moc perspective it's nothing different the private just makes it not invokable directly from other parts of C++.


> On June 18, 2014, 1 p.m., David Edmundson wrote:
> > shell/shellcorona.cpp, line 1232
> > <https://git.reviewboard.kde.org/r/118812/diff/1/?file=282339#file282339line1232>
> >
> >     just QCoreApplication::exit()
> 
> Martin Gräßlin wrote:
>     I think it makes sense to have the error code. This could allow startkde to abort the startup.

QCoreApplication::exit(1) then.

I meant we don't need the instance() part.

It's possible this will fix your issue?


- David


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


On June 18, 2014, 11:36 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118812/
> -----------------------------------------------------------
> 
> (Updated June 18, 2014, 11:36 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> [plasmashell] Show a warning if there are no Shaders and exit
> 
> If there are no Shaders Plasma doesn't work. If we detect this we show
> a warning (without GL) and exit.
> 
> This doesn't really work as Qt has a bug which doesn't allow to detect
> whether Shaders are supported and the exit just doesn't work.
> 
> 
> Diffs
> -----
> 
>   shell/shellcorona.h f500e837b5957e14e70ac4b24da0cdf7970a7171 
>   shell/shellcorona.cpp 4abe3432f30a8c4eb90806893f15c7c50f0e1ac2 
> 
> Diff: https://git.reviewboard.kde.org/r/118812/diff/
> 
> 
> Testing
> -------
> 
> With Mesa drivers:
> LIBGL_ALWAYS_INDIRECT=1 plasmashell
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140618/13b0c5fa/attachment.html>


More information about the Plasma-devel mailing list