D6724: Replace Q_FOREACH with C++11 range-for

Nicolas Fella noreply at phabricator.kde.org
Sat Jul 15 19:02:19 UTC 2017


nicolasfella added a comment.


  > except for the one in daemon.cpp in deviceIdByName which might have a bug / typo in it
  
  This seems to be just really bad naming. d is also the name of a member variable. The Device* should be renamed to avoid such confusion

REPOSITORY
  R224 KDE Connect

REVISION DETAIL
  https://phabricator.kde.org/D6724

To: jeanv, #kde_connect, nicolasfella
Cc: nicolasfella, tfella, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, apol, hkaelberer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170715/41133d77/attachment.html>


More information about the KDEConnect mailing list