Immediately try connecting to the mesh after a succesful join.

[meshlink] / TODO

diff --git a/TODO b/TODO
index 3d0a7c8c637cce76437a8752f521b57bf1410fc0..210c2a3f0419d51c50b584eabd82e7c1e9ff19b4 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,15 +1,24 @@
-Things left to do to make cabal superstable:
---------------------------------------------
- * Check for connection loops. Inter-daemon
-   connections should always satisfy the
-   tree property.
- * Check for duplicates. If there's a
-   timeout and a host reconnects before the
-   old connection is closed, duplicate
-   entries appear in every connection list.
- * Redundancy: multiple ConnectTo lines, if
-   one fails others might be tried.
- * Persistence: don't quit when no connection
-   can be made directly after start of the
-   daemon.
+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:
+
+* 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
+
+After that is completed:
+
+* Add meshlink_handle_t *mesh as the first argument to functions that need it
+
+Then:
+
+* Write code for all the functions in meshlink.c (some can be moved from libmeshlink.c)
+
+Other tasks:
+
+* Use snprintf() instead of xasprintf() when generating filenames
+  - Use char filename[PATH_MAX] on the stack. No more need to free().