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

Saikrishna Arcot saiarcot895 at gmail.com
Sun Apr 5 03:03:07 UTC 2015


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

(Updated April 5, 2015, 3:03 a.m.)


Review request for kdeconnect.


Changes
-------

Add support for payloads of unknown size.


Repository: kdeconnect-kde


Description (updated)
-------

When sending a payload,  a `===BEGIN PAYLOAD===` and `===END PAYLOAD===` markers are sent with the payload. On the receiving end, the data between these two markers is captured, and the size is checked. If the size doesn't match, then the current implementation just returns a package without a payload (which might be good or bad). If the size is unknown, then the data is assumed to have been transmitted correctly.

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/backends/bluetooth/CMakeLists.txt 125fa87a825b056395a8ce5ef0298665fd2e6293 
  core/backends/bluetooth/bluetoothdevicelink.h 199d9ee4c6b89065154e82b6fcd2cea204c0ef31 
  core/backends/bluetooth/bluetoothdevicelink.cpp e3c1e3335a312a2b9289a7806e6a4d9c9174c73c 
  core/backends/bluetooth/bluetoothdownloadjob.h PRE-CREATION 
  core/backends/bluetooth/bluetoothdownloadjob.cpp PRE-CREATION 
  core/backends/devicelinereader.h a5255c77d95c13e5f806576bac2697fb4bc94708 
  core/backends/devicelinereader.cpp bba0bdaae878be95680aa3c84aed8df9d9b81a8a 
  core/filetransferjob.cpp 66866906a6509ebb0ba00c1b48647c5807262120 

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


Testing (updated)
-------

Android to KDE: Tested, and works. Sent a 1.1 MB picture from Android to KDE.

KDE to Android: Works for smaller file sizes.


Thanks,

Saikrishna Arcot

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


More information about the KDEConnect mailing list