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

[Saikrishna Arcot]

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

(Updated May 3, 2015, 8:19 p.m.)


Review request for kdeconnect.


Changes
-------

Rework payload code. This _should_ also allow displaying the progress of the file transfer.


Repository: kdeconnect-kde


Description (updated)
-------

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 (updated)
-----

  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 (updated)
-------

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/3bcde927/attachment.html>