----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126796/#review92255 -----------------------------------------------------------
src/qt-oauth-lib/oauth2.cpp (line 81) <https://git.reviewboard.kde.org/r/126796/#comment62926> if (!strAccessTokenFromSettings.isEmpty()) src/qt-oauth-lib/oauth2.cpp (line 98) <https://git.reviewboard.kde.org/r/126796/#comment62927> constify: QString OAuth2::getClientID() const src/qt-oauth-lib/oauth2.cpp (line 107) <https://git.reviewboard.kde.org/r/126796/#comment62928> const src/qt-oauth-lib/oauth2.cpp (line 127) <https://git.reviewboard.kde.org/r/126796/#comment62929> m_strAccessToken.clear(); src/qt-oauth-lib/oauth2.cpp (line 159) <https://git.reviewboard.kde.org/r/126796/#comment62930> use the .isEmpty() QString method src/qt-oauth-lib/oauth2.cpp (line 165) <https://git.reviewboard.kde.org/r/126796/#comment62931> const src/utils/freesound.cpp (line 310) <https://git.reviewboard.kde.org/r/126796/#comment62932> Keep the QProcess->start(exec, QStringList args) format src/utils/resourcewidget.cpp (line 742) <https://git.reviewboard.kde.org/r/126796/#comment62933> Here, we create and leak the networkManager on each call to slotAccessTokenReceived. There should be one private networkmanager declared in header file and used on each call, deleted when ResourceWidget is deleted src/utils/resourcewidget.cpp (line 756) <https://git.reviewboard.kde.org/r/126796/#comment62934> The QNetworkReply pointer should be deleted in the DownloadRequestFinished slot with something like this: reply->deleteLater(); - Jean-Baptiste Mardelle On fév. 11, 2016, 9:51 matin, Roger Morton wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126796/ > ----------------------------------------------------------- > > (Updated fév. 11, 2016, 9:51 matin) > > > Review request for Kdenlive and Vincent Pinon. > > > Repository: kdenlive > > > Description > ------- > > online resource -logs into users freesound accounts to get permission for HQ > audio. Fixes archive.org video search new dependancy KF5WebKit. > (libkf5webkit5-dev) > > > Diffs > ----- > > src/CMakeLists.txt 5b7fd58 > src/qt-oauth-lib/CMakeLists.txt PRE-CREATION > src/qt-oauth-lib/logindialog.h PRE-CREATION > src/qt-oauth-lib/logindialog.cpp PRE-CREATION > src/qt-oauth-lib/oauth2.h PRE-CREATION > src/qt-oauth-lib/oauth2.cpp PRE-CREATION > src/ui/freesound_ui.ui 55a453d > src/ui/logindialog_ui.ui PRE-CREATION > src/utils/abstractservice.h 0a6a4a7 > src/utils/archiveorg.h 5cdc84f > src/utils/archiveorg.cpp 1be4d6f > src/utils/freesound.h 27eefad > src/utils/freesound.cpp d2b3c0e > src/utils/openclipart.h 7a4df1e > src/utils/openclipart.cpp adbba68 > src/utils/resourcewidget.h c171e3c > src/utils/resourcewidget.cpp ddbfebe > > Diff: https://git.reviewboard.kde.org/r/126796/diff/ > > > Testing > ------- > > Built. Tested file downloads from freesound. Auhenticates to freesound. keeps > authenticated until you delete $HOME/.config/kde.org/kdenlive.conf > Archive.org video search is now working. Display animiated .gif as the > preview file from Archive.org videos. > No longer have to click search button twice to get it to do the search the > first time it runs > Option to not use personal freesound account and just use HQ preview file > instead. > Downloads low quality preview file in background and does not notify you of > it finishing. > > > File Attachments > ---------------- > > tokens to settings patch > > https://git.reviewboard.kde.org/media/uploaded/files/2016/02/04/9529c467-2fa8-459e-84c5-a899ca4275ee__0002-move-token-storage-in-freesound-to-settings-file.patch > > > Thanks, > > Roger Morton > >
_______________________________________________ kdenlive mailing list kdenlive@kde.org https://mail.kde.org/mailman/listinfo/kdenlive