Review Request 123094: Add Bluetooth support for sending and receiving payloads. (KDE)

Saikrishna Arcot saiarcot895 at gmail.com
Sun May 3 22:46:37 UTC 2015


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

(Updated May 4, 2015, 12:46 a.m.)


Review request for kdeconnect.


Summary (updated)
-----------------

Add Bluetooth support for sending and receiving payloads. (KDE)


Repository: kdeconnect-kde


Description
-------

When sending a payload, a randomly-generated UUID is sent with the payload. The payload is sent on a separate Bluetooth connection through that UUID, with the sending device acting as the server and the receiving device acting as the receiver.

In addition, `core/filetransferjob.cpp` was edited to listen to the `aboutToClose()` signal instead of the `disconnected()` signal, which is guaranteed to exist on all implementations of `QIODevice`. I'm not sure of the side effects for the LAN device link.


Diffs
-----

  core/filetransferjob.cpp 66866906a6509ebb0ba00c1b48647c5807262120 
  core/backends/devicelinereader.h a5255c77d95c13e5f806576bac2697fb4bc94708 
  core/backends/bluetooth/bluetoothuploadjob.cpp PRE-CREATION 
  core/backends/bluetooth/bluetoothuploadjob.h PRE-CREATION 
  core/backends/bluetooth/bluetoothdownloadjob.cpp PRE-CREATION 
  core/backends/bluetooth/bluetoothdownloadjob.h PRE-CREATION 
  core/backends/bluetooth/bluetoothdevicelink.cpp e3c1e3335a312a2b9289a7806e6a4d9c9174c73c 
  core/backends/bluetooth/CMakeLists.txt 125fa87a825b056395a8ce5ef0298665fd2e6293 
  core/backends/bluetooth/bluetoothdevicelink.h 199d9ee4c6b89065154e82b6fcd2cea204c0ef31 

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


Testing
-------

To be tested (the Android side has to be updated first).


Thanks,

Saikrishna Arcot

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


More information about the KDEConnect mailing list