> On Nov. 2, 2015, 9:52 p.m., Albert Astals Cid wrote: > > 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
QTabWidget::currentChanged probably because the tab widget (KPart widget) is destroyed before QTabWidget. delete m_tabWidget in Shell destructor works too. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125890/#review87890 ----------------------------------------------------------- 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
