Commit Graph

13 Commits

Author SHA1 Message Date
Gregory Mullen (grayhatter) 1f94c96266 Make the friend message callback stateless 2016-08-20 11:07:49 -07:00
iphydf 6935643f9a Fix some compiler warnings. 2016-08-18 17:10:15 +01:00
iphydf dd8a568141 Make self_connection_status callback stateless.
**What are we doing?**

We are moving towards stateless callbacks. This means that when registering a
callback, you no longer pass a user data pointer. Instead, you pass a user data
pointer to tox_iterate. This pointer is threaded through the code, passed to
each callback. The callback can modify the data pointed at. An extra indirection
will be needed if the pointer itself can change.

**Why?**

Currently, callbacks are registered with a user data pointer. This means the
library has N pointers for N different callbacks. These pointers need to be
managed by the client code. Managing the lifetime of the pointee can be
difficult. In C++, it takes special effort to ensure that the lifetime of user
data extends at least beyond the lifetime of the Tox instance. For other
languages, the situation is much worse. Java and other garbage collected
languages may move objects in memory, so the pointers are not stable. Tox4j goes
through a lot of effort to make the Java/Scala user experience a pleasant one by
keeping a global array of Tox+userdata on the C++ side, and communicating via
protobufs. A Haskell FFI would have to do similarly complex tricks.

Stateless callbacks ensure that a user data pointer only needs to live during a
single function call. This means that the user code (or language runtime) can
move the data around at will, as long as it sets the new location in the
callback.

**How?**

We are doing this change one callback at a time. After each callback, we ensure
that everything still works as expected. This means the toxcore change will
require 15 Pull Requests.
2016-08-17 14:57:20 +01:00
irungentoo 8e80ced6ce Move savedata to options struct.
Add a way to select the type of savedata (normal savedata, load a
secret key, potentially others?) to load.
2015-05-22 18:23:56 -04:00
irungentoo 782d68cdc3 Syncbot fixes. 2015-05-20 14:09:05 -04:00
irungentoo d711362622 Merged normal message function and send action function into one
messaging function.

This removes code duplication and allows us to easily add new message
types to the api without having to add new functions.
2015-03-18 15:32:53 -04:00
Dubslow 3bf259f69d Convert core code depending on the api 2015-03-12 18:23:14 -05:00
irungentoo d83efd35dd Ported some of the code in testing/ to the new api. 2015-02-24 20:29:01 -05:00
irungentoo 534efc7349 Fixed irc syncbot. 2014-11-10 18:03:09 -05:00
dubslow ca38ee8bc7 fix #1120 2014-10-17 06:04:08 -05:00
irungentoo 9ef1be1b6f Use monotonic time function already in toxcore. 2014-10-15 13:28:19 -04:00
irungentoo 3d1c0f0536 irc_syncbot now parse out NULL characters from the Tox groupchat. 2014-10-15 09:01:00 -04:00
irungentoo 6985353392 Added basic IRC group syncbot to testing. 2014-10-14 19:49:29 -04:00