----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116741/#review55858 -----------------------------------------------------------
src/services/jamendo/JamendoService.cpp <https://git.reviewboard.kde.org/r/116741/#comment38903> The formatting looks wrong over here. Is this intended? - Vishesh Handa On March 11, 2014, 8:45 p.m., yash ladia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116741/ > ----------------------------------------------------------- > > (Updated March 11, 2014, 8:45 p.m.) > > > Review request for Amarok. > > > Bugs: 331934 > https://bugs.kde.org/show_bug.cgi?id=331934 > > > Repository: amarok > > > Description > ------- > > I have corrected the URL to the new one. > Also, when there was an error in downloading the relatively large (17.3 mb) > file, it just used to quit silently. With my patch it still does, but the > "update" button becomes enabled so that the download can be attempted again > without restarting Amarok. I had to make sure that the 'm_listDownloadJob = > 0;' (line 205) was called so I placed it before the return statement. > Otherwise, it was giving a segfault error when Amarok was being quit. > > > Diffs > ----- > > src/services/jamendo/JamendoService.cpp e4a7ecc > src/services/jamendo/JamendoXmlParser.h 6bdc67d > > Diff: https://git.reviewboard.kde.org/r/116741/diff/ > > > Testing > ------- > > Jamendo works now. File downloads as expected. > Also, I testing the working of the new update button behavior by running the > update without any internet connection. The "update" button became enabled in > this case. > > > Thanks, > > yash ladia > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
