> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel