----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101310/#review3416 -----------------------------------------------------------
Ship it! fix the indentation and commit :) mainwindow.cpp <http://git.reviewboard.kde.org/r/101310/#comment2875> needs to be indented 4 spaces - Aaron J. On May 10, 2011, 3:08 p.m., Giorgos Tsiapaliwkas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101310/ > ----------------------------------------------------------- > > (Updated May 10, 2011, 3:08 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 8d47b09 > 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
