> On June 20, 2016, 10:27 p.m., Matěj Laitl wrote: > > src/musicbrainz/MusicDNSAudioDecoder.cpp, line 226 > > <https://git.reviewboard.kde.org/r/128248/diff/1/?file=469271#file469271line226> > > > > This looks like an unrelated change. Please strip of a probably post as > > a separate patch. Applies also to the change few lines below.
Do I commit separately or do i open a review request? - Olivier ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128248/#review96755 ----------------------------------------------------------- On June 19, 2016, 11:44 a.m., Olivier Churlaud wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128248/ > ----------------------------------------------------------- > > (Updated June 19, 2016, 11:44 a.m.) > > > Review request for Amarok and Myriam Schweingruber. > > > Bugs: 349313 > https://bugs.kde.org/show_bug.cgi?id=349313 > > > Repository: amarok > > > Description > ------- > > I removed every possibility to use non SSL links > > > Diffs > ----- > > src/context/applets/wikipedia/WikipediaApplet.cpp 2ceb2b0 > src/context/applets/wikipedia/wikipediaGeneralSettings.ui 84cb5df > src/context/engines/wikipedia/WikipediaEngine.cpp 969d8fc > src/musicbrainz/MusicDNSAudioDecoder.cpp ea39a1b > > Diff: https://git.reviewboard.kde.org/r/128248/diff/ > > > Testing > ------- > > It compiles. However not tested, because make install always install in > /usr/local, whatever I give in -DCMAKE_INSTALL_PREFIX. And I don't want to > mess my install. > > > Thanks, > > Olivier Churlaud > >
