[Kde-hardware-devel] Review Request 126342: Fixed Clazy warning [-Wclazy-function-args-by-ref] and [-Wclazy-reserve-candidates]

[Artur Puzio]

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

(Updated Dec. 14, 2015, 6:38 p.m.)


Status
------

This change has been marked as submitted.


Review request for Bluedevil and David Rosca.


Changes
-------

Submitted with commit 5c581d4dd834409dc0bd57d03dd7573dd9b74c07 by David Rosca on behalf of Artur Puzio to branch master.


Repository: bluez-qt


Description
-------

Fixed `warning: Missing reference on non-trivial type (*some*Ptr) [-Wclazy-function-args-by-ref]` - without touching headers without `_p` prefix or not in tests.
Fixed `warning: Reserve candidate [-Wclazy-reserve-candidates]`
Fixed Also one warning: warning: non-POD static (QString) and one warning: QString(QLatin1String) in tests


Diffs
-----

  autotests/fakebluez/deviceinterface.cpp 2f58f00 
  autotests/inputtest.h ff8e8c4 
  autotests/inputtest.cpp 7955ee6 
  autotests/mediaplayertest.h 7427fbd 
  autotests/mediaplayertest.cpp 230b949 
  src/adapter_p.h ab54315 
  src/adapter_p.cpp 33fe048 
  src/device_p.h ec1cdcd 
  src/device_p.cpp 2490b0c 
  src/imports/declarativeadapter.cpp 203a13b 
  src/imports/declarativemanager.cpp 634767c 
  src/manager_p.h 77cae10 
  src/manager_p.cpp a343921 
  src/obextransfer.cpp ffa3cdb 
  src/obextransfer_p.h 780755d 
  src/pendingcall.cpp cdd671b 
  src/utils.cpp 3e1e7c0 
  tests/adaptersreceiver.h 3997906 
  tests/adaptersreceiver.cpp 4ef0558 
  tests/devicereceiver.h d0f7072 
  tests/devicereceiver.cpp c6dde51 

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


Testing
-------

Automated tests passed.


Thanks,

Artur Puzio

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20151214/9bf779fc/attachment.html>