Review Request 127458: Fix for warnings, one of them a probable bug

[Stefano Pettini]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127458/
-----------------------------------------------------------

(Updated Aug. 20, 2016, 1:01 p.m.)


Status
------

This change has been marked as submitted.


Review request for Amarok.


Changes
-------

Submitted with commit 970e985d6100070aa105dd602da8c294687385ac by Mat?j Laitl on behalf of Stefano Pettini to branch master.


Repository: amarok


Description
-------

This fixes some minor C++ warnings: one unused variable and another one about operator precedence. However I think the last one was I bug, that I think I fixed. It was about bpm in SqlScanProcessor. In addition, some warnings regarding deprecated functions are addressed.


Diffs
-----

  src/configdialog/dialogs/DatabaseConfig.cpp 1cb2d97 
  src/core-impl/collections/db/sql/SqlScanResultProcessor.cpp ea9c911 
  src/core-impl/storage/sql/mysqlserverstorage/MySqlServerStorageFactory.cpp 5cdb693 
  src/services/mp3tunes/harmonydaemon/Mp3tunesHarmonyDaemon.cpp 5b09f07 
  src/services/mp3tunes/libmp3tunes/harmony.c ead0345 
  src/statemanagement/DefaultApplicationController.cpp 5886789 

Diff: https://git.reviewboard.kde.org/r/127458/diff/


Testing
-------


Thanks,

Stefano Pettini

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20160820/02fdf3df/attachment.html>