X-Git-Url: http://git.meshlink.io/?a=blobdiff_plain;f=TODO;h=c52751c771f8c363e5357feddac351d3a914dcb9;hb=refs%2Fheads%2Fchannels;hp=210c2a3f0419d51c50b584eabd82e7c1e9ff19b4;hpb=f216909958888bb5549ca9beeb5fec61e6017860;p=meshlink

diff --git a/TODO b/TODO
index 210c2a3f..c52751c7 100644
--- a/TODO
+++ b/TODO
@@ -1,24 +1,51 @@
 TODO list for MeshLink
 ----------------------
 
-Refactoring: getting rid of global variables. Ultimately, no global variables
-should be used by the library (if possible). Tinc uses quite a lot of global
-variables. Refactoring this should go into steps:
+* Speed up start: immediately do outgoing connections instead of waiting 5 seconds.
 
-* Temporarily create a global meshlink_handle_t *mesh (done)
-* Move global variables to meshlink_handle_t (partially done)
-  - This means moving the declaration from the old .h file to meshlink_internal.h inside meshlink_handle_t
-  - Then doing sed 's/variable/mesh->&/g' *.c -i, and running make to check for any errors
+* meshlink_join():
+  - add checks that we only join another mesh if we did not connect to any other node before.
+  - only allow meshlink_join() when the library thread is not running.
+  - remove old host config file if the invitation gave the node a different name.
+  - how to handle nodes that are part of one mesh and want to join another?
 
-After that is completed:
+* meshlink_leave()?
+  - leaving a mesh is basically starting all over again, with new private keys?
 
-* Add meshlink_handle_t *mesh as the first argument to functions that need it
+* Allow meshlink_open() to be called with a NULL name, in anticipation of meshlink_join().
+  - Do not allow meshlink_start() if no Name is set.
 
-Then:
+* Add a logging callback.
+  - One in a meshlink_handle_t for everything that happens after meshlink_open().
+  - Add a global one for library calls that don't involve a valid meshlink_handle_t *.
+  - Get rid of "priority".
 
-* Write code for all the functions in meshlink.c (some can be moved from libmeshlink.c)
+* Fill in mesh->errno whenever an error happens (= when a function returns false or NULL).
 
-Other tasks:
+* Add autoconf/automake stuff to call Doxygen.
 
-* Use snprintf() instead of xasprintf() when generating filenames
-  - Use char filename[PATH_MAX] on the stack. No more need to free().
+* Write a manual
+  - Document API
+  - Explain what MeshLink does and how it should be used in the application
+  - Write simple examples for common operations
+  - List possible ways of synchronisation between application and library threads
+    - simple polling
+    - pthread_cond?
+    - pipe() to signal an event loop of the application's choice
+    - whatever equivalent(s) there are on Windows
+
+* Add a "channel" library to MeshLink.
+  - Support multiple TCP- and UDP-like streams between two nodes.
+
+* Move queue code to its own source file.
+
+Not finished but being worked on:
+---------------------------------
+
+* Provide thread-safety to functions that return pointers to meshlink_node_t.
+  - The mesh->nodes tree can be updated by the library thread at any time,
+    so it must be protected by a mutex or rwlock.
+  - Individial node_t's must never be freed or moved except in meshlink_close().
+  - Check all public API functions.
+
+* Write a test suite for the library.