> On March 30, 2014, 11:28 a.m., Lamarque Souza wrote: > > By the way, there must be a "delete installer;" before the mentioned > > "return 1" line, otherwise we will have a memory leak.
Also the installer is deleted finally at the end when successful. - Andrei ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117175/#review54586 ----------------------------------------------------------- On March 30, 2014, 12:28 p.m., Andrei Amuraritei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117175/ > ----------------------------------------------------------- > > (Updated March 30, 2014, 12:28 p.m.) > > > Review request for KDE Runtime, Albert Astals Cid, Aaron J. Seigo, Ian > Monroe, and Lamarque Souza. > > > Bugs: 306279 and 325028 > http://bugs.kde.org/show_bug.cgi?id=306279 > http://bugs.kde.org/show_bug.cgi?id=325028 > > > Repository: kde-runtime > > > Description > ------- > > When installing a new .comic provider from GHNS, it doesn't appear in the > installed list on the comic widget. > This fixes it. > > > Diffs > ----- > > plasma/tools/plasmapkg/main.cpp 61492fe > > Diff: https://git.reviewboard.kde.org/r/117175/diff/ > > > Testing > ------- > > Compile, add new comic widget, install new comic providers. > > > File Attachments > ---------------- > > patch with diff > > https://git.reviewboard.kde.org/media/uploaded/files/2014/03/30/313c693a-4c1e-4def-9c8f-4ab949d01874__kde-runtime-diff.patch > > > Thanks, > > Andrei Amuraritei > >