tox4j / deprecated-tox4j

Deprecated, see http://github.com/TokTok/jvm-toxcore-c
http://github.com/TokTok/jvm-toxcore-c
Other
42 stars 17 forks source link

Remote side cancelling file transfer request crashes Antox #31

Closed TheLastProject closed 9 years ago

TheLastProject commented 9 years ago

The cancelling was done from qTox, if that matters.

Same issue on Antox' repo: subliun/Antox#26

sonOfRa commented 9 years ago

The problem with this is our "in-between" layer. We use the new tox api (it was developed as part of tox4j), and we wrote a glue layer to express the new api in terms of the old api. This didn't work properly for file transfers. My motivation to fix this at the moment is rather low, as irungentoo is working on implementing the new API in the core repository, at which point our compat layer will be deleted.

iphydf commented 9 years ago

The compat layer is now gone. I will soon do some more testing with this and close the bug once I can confirm it works.

iphydf commented 9 years ago

This now works. Reopen if it still happens for you.