----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124121/#review81558 -----------------------------------------------------------
Ship it! src/kplotwidget.h (line 188) <https://git.reviewboard.kde.org/r/124121/#comment55902> might be good to explain that KPlotWidget will take ownership (unless set otherwise) - David Edmundson On June 18, 2015, 9:59 a.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124121/ > ----------------------------------------------------------- > > (Updated June 18, 2015, 9:59 a.m.) > > > Review request for KDE Frameworks and Christoph Feck. > > > Repository: kplotting > > > Description > ------- > > KPlotWidget: add setAutoDeletePlotObjects, fix memory leak in > replacePlotObject > > > Diffs > ----- > > src/kplotwidget.h e5b3e16b31d6d34b7c544df42531f2f91105830d > src/kplotwidget.cpp 8dcc4ecef798a53376dd8cc4a0a6e6c1804db593 > > Diff: https://git.reviewboard.kde.org/r/124121/diff/ > > > Testing > ------- > > (Not trivial to unittest because KPlotObject is not a QObject.) > > > Thanks, > > David Faure > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel