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