D17164: Fix container detach warnings from clazy
Nicolas Fella
noreply at phabricator.kde.org
Mon Nov 26 12:20:30 GMT 2018
nicolasfella added inline comments.
INLINE COMMENTS
> apol wrote in kdeconnect-cli.cpp:156
> Why didn't you use qAsConst here?
According to http://doc.qt.io/qt-5/qtglobal.html#qAsConst qAsConst only works for lvalues, not rvalues, so it can't be used directly on the result of a method call.
REPOSITORY
R224 KDE Connect
BRANCH
nodetach
REVISION DETAIL
https://phabricator.kde.org/D17164
To: nicolasfella, #kde_connect, apol
Cc: apol, kdeconnect, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181126/573c489a/attachment.html>
More information about the KDEConnect
mailing list