Review Request 125997: Catch other openGL error gracefully
David Edmundson
david at davidedmundson.co.uk
Mon Nov 9 11:51:40 UTC 2015
> On Nov. 9, 2015, 6:57 a.m., Martin Gräßlin wrote:
> > Could you please explain the change? Reduce string comparison?
> >
> > Anyway: we should probably extend the checks to also include the EGL/XCB error messages an EGL/Wayland error messages
there's a second
if message == "blah"
which is the main important change
then I added the if Q_UNLIKELY to potentially speed things up as 99.9999% of the time it will be debug not fatal messages.
then I reformatted it so it was a bit cleaner as I ended up 4 brackets deep
- David
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125997/#review88172
-----------------------------------------------------------
On Nov. 8, 2015, 9:07 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125997/
> -----------------------------------------------------------
>
> (Updated Nov. 8, 2015, 9:07 p.m.)
>
>
> Review request for Plasma.
>
>
> Repository: plasma-workspace
>
>
> Description
> -------
>
> CCBUG: 354966
> CCBUG: 355054
>
>
> Diffs
> -----
>
> shell/main.cpp a774cb4e8d18032ff3fd39892b8171b96a94cc92
>
> Diff: https://git.reviewboard.kde.org/r/125997/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151109/dd730fe1/attachment.html>
More information about the Plasma-devel
mailing list