----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101310/#review3183 -----------------------------------------------------------
for the crash, if not creating m_central again on line 226 doesn't fix it, send a backtrace to the mailing list or find me on irc and we can look at it together. mainwindow.cpp <http://git.reviewboard.kde.org/r/101310/#comment2711> this shouldn't be needed; m_central is already created at this point. perhaps this is even the source of the crash? mainwindow.cpp <http://git.reviewboard.kde.org/r/101310/#comment2712> this isn't needed as we already know it isn't the first start ;) - Aaron J. On May 6, 2011, 6:36 p.m., Giorgos Tsiapaliwkas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101310/ > ----------------------------------------------------------- > > (Updated May 6, 2011, 6:36 p.m.) > > > Review request for Plasma. > > > Summary > ------- > > hello, > > this patch adds a menu bar for startpage at file.(Go to file->startpage).The > actual work is that using the startpage button the user is able to go back > to the startpage from the editor. > > Unfortunately,there is a regression.If you go to start page using the > startpage button when you will load a project from the recent list you will > receive a sigsegment error.I think that the problem is located at the > editors/edipage.cpp:line 152 (emit loadEditor(offers, path);). > > thank you for your time. > > > Diffs > ----- > > mainwindow.h ea78e4d > mainwindow.cpp bb241a0 > plasmateui.rc 05aa1cd > > Diff: http://git.reviewboard.kde.org/r/101310/diff > > > Testing > ------- > > tseting done by me > > > Thanks, > > Giorgos > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
