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

Stefano Pettini stefano.pettini at gmail.com
Mon Mar 21 23:03:07 UTC 2016


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

Review request for Amarok.


Repository: amarok


Description
-------

This fixes two 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


Diffs
-----

  src/configdialog/dialogs/DatabaseConfig.cpp 1cb2d97 
  src/core-impl/collections/db/sql/SqlScanResultProcessor.cpp ea9c911 

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/20160321/7b00bca3/attachment.html>


More information about the Amarok-devel mailing list