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

[Aleix Pol Gonzalez]

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


+1 looks good.

Please, wait for the maintainer to give a final decision though.

- Aleix Pol Gonzalez


On Dec. 14, 2015, 11:44 a.m., Artur Puzio wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126342/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2015, 11:44 a.m.)
> 
> 
> Review request for Bluedevil and David Rosca.
> 
> 
> 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.
> Also one warning: warning: non-POD static (QString) and 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 
>   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/850ec0fa/attachment.html>