----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125890/#review87890 -----------------------------------------------------------
I'd prefer to know who decides to set a tab of -1 since this doesn't seem to be needed in the non frameworks branch seems like the code is "breaking" somewhere else - Albert Astals Cid On oct. 31, 2015, 11:52 a.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125890/ > ----------------------------------------------------------- > > (Updated oct. 31, 2015, 11:52 a.m.) > > > Review request for Okular. > > > Repository: okular > > > Description > ------- > > Don't try to set invalid (-1) active tab. > Add the same check also in removeTab. > > > Diffs > ----- > > shell/shell.cpp ce5e76c > > Diff: https://git.reviewboard.kde.org/r/125890/diff/ > > > Testing > ------- > > No longer crashes. > > > Thanks, > > David Rosca > >
_______________________________________________ Okular-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/okular-devel
