From 3ac6b440e4570b82dee7e1e033d8f8ec4fe34f13 Mon Sep 17 00:00:00 2001 From: Matteo Cypriani Date: Wed, 8 Jun 2011 10:56:14 +0200 Subject: [PATCH] Update TODOs --- TODO | 17 +++++++++-------- owlps-positioning/TODO | 16 +++++++++------- 2 files changed, 18 insertions(+), 15 deletions(-) diff --git a/TODO b/TODO index a20adb3..6426270 100644 --- a/TODO +++ b/TODO @@ -6,8 +6,9 @@ ° Translate comments & help. - Use string for network exchanges? - Mark arguments as const in function headers if needed - That is done in the owlps-positioning C++ code, but not constantly - in C modules. [Done in libowlps & libowlps-client.] + That is (mostly) done in the owlps-positioning C++ code, but not + constantly in C modules. + [Done in libowlps & libowlps-client.] - Allow to use hostnames instead of IP addresses in all modules getaddrinfo(3) (or gethostbyname(3)) should be used when opening sockets in libowlps, then string buffer size have to be adapted in @@ -29,18 +30,18 @@ * Aggregator -- inet_ntoa() is not secure with threads - (But it is currently used by only one thread.) Use inet_ntop() - instead? - Use locks to read/write the AP list Currently, several threads can access the list simultaneously, and that's not cool! +- inet_ntoa() is not secure with threads + (But it is currently used by only one thread.) Use inet_ntop() + instead? - Refactor got_request(). - Use the type of a request to identify it? (along with the mobile MAC address and the request time) - got_request(): option for the maximal difference time - For implicit packet, we consider that packet from the same MAC and - received within an interval of 10ms are part of the same + For implicit packets, we consider that packets from the same MAC + and received within an interval of 10ms are part of the same request. We should define an option to allow user to choose the time he wants. - Allow blank parameters for output options. @@ -54,7 +55,7 @@ Currently the size is corrected only for data packets. - Move endianess #defines in libowlps? - Merge Makefile and Makefile_atheros? - Use autohell? + Use autohell, cmake, etc.? - Use positive (USE_CONFUSE) and negative (NO_USE_PTHREAD) options? The default options could be modified, but that's not so important, is it? diff --git a/owlps-positioning/TODO b/owlps-positioning/TODO index f61eaa2..1134025 100644 --- a/owlps-positioning/TODO +++ b/owlps-positioning/TODO @@ -7,6 +7,8 @@ streams (no need to cast each time anymore). - User interface + ° Verbose and/or debug level (should replace #ifdef DEBUG): print + the options, etc. ° Case-insensitive string comparison (for algorithm names, etc.). ° Use a prefix for configuration files (search for config files set with relative path in owlps-positioning.cfg in the same directory). @@ -38,12 +40,12 @@ - MinMax ° Différencier le pas pour X, Y et Z ? -- Renommages de membres - ° InputMedium : - °° current_line_nb et get_current_line_nb() +- Members renaming + ° InputMedium: + °° current_line_nb & get_current_line_nb() °° get_next_request() −> read_next_request() - ° Input : get_next_request() −> read_next_request() - ° Area : p_min et p_max −> coord_min et coord_max + ° Input: get_next_request() −> read_next_request() + ° Area: p_min et p_max −> coord_min et coord_max - « C++ en action » ° Espaces de noms ? 109 @@ -59,8 +61,8 @@ respecter ce principe en utilisant mieux les opérateurs et en ne proposant pas d'accesseur direct. -- Divers - ° Passage de pointeurs en argument : const * const +- Miscellaneous + ° Pointer function arguments: const * const - Timestamp Dans les tests, il y a une probabilité de 1/1000000 que la valeur