Review Request 116647: Some cleaning in lanlinkprovider.cpp

Àlex Fiestas afiestas at kde.org
Fri Mar 7 01:12:57 UTC 2014


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

Review request for kdeconnect.


Repository: kdeconnect-kde


Description
-------

While learning this code I have been doing some small cleaning, mostly:
-Return early in some cases
-add m to member variables
-spaces after coma (foo, bar, x)
-Fix in how to handle hasPendingConnections (it might be more than one, so we have to do while)
-Fix memory leak on connectionError
-Some renames in variables that were hard to understand for a newbie (like me), mostly np to receivedPackage, id to deviceID
-Keep an instance of QUdpSocket instead of instancing one every time we want to send datagrams


Diffs
-----

  kded/backends/lan/lanlinkprovider.h 85938b2 
  kded/backends/lan/lanlinkprovider.cpp a205e1f 

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


Testing
-------

Connected both ways, device to pc, and pc to device.
Ping works
MPRIS works
Sending image and video, works.


Thanks,

Àlex Fiestas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20140307/baf6ad47/attachment.html>


More information about the KDEConnect mailing list