From e8332fd0f66afd26bf62a80cc620e5219f976418 Mon Sep 17 00:00:00 2001 From: Matteo Cypriani Date: Thu, 7 Apr 2011 10:18:06 +0200 Subject: [PATCH] Update TODOs --- TODO | 21 ++++++++++++++------- owlps-positioning/TODO | 7 ------- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/TODO b/TODO index dc31ebc..970004d 100644 --- a/TODO +++ b/TODO @@ -5,7 +5,10 @@ - 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.] -- Allow to use hostnames instead of IP addresses. +- 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 + all modules to be able to contain a long hostname. - Use struct ether_addr to store MAC addresses? We could use the struct ether_addr to store binary MAC addresses, and convert them to strings with ether_ntoa() instead of @@ -14,14 +17,18 @@ * libowlps -- Delete timestamp_is_null()? - This function is currently not used. +- Currently unused functions: + ° owl_timestamp_is_null() + ° owl_iface_channel_hop() + ° owl_iface_set_channel() (except by owl_iface_channel_hop()) +- Move owl_iface_mode_monitor() in owlps-listenerd.c. * Aggregator - inet_ntoa() is not secure with threads - Use inet_ntop() instead? + (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! @@ -34,14 +41,14 @@ request. We should define an option to allow user to choose the time he wants. - Allow blank parameters for output options. -- Fix segfault on SIGINT. * Listener +- Refactor and review read_packet(). +- Listen for autocalibration requests without sending requests? - read_packet(): use ieee80211_header_size for all implicit packets Currently the size is corrected only for data packets. -- Fix segfault when rtap_iface is not in monitor mode (?) on Foneras. - Move endianess #defines in libowlps? - Merge Makefile and Makefile_atheros? Use autohell? @@ -52,7 +59,7 @@ * Client -- Allow to use a string for the direction +- Allow to use a string for the direction? Could be nice, but probably useless. diff --git a/owlps-positioning/TODO b/owlps-positioning/TODO index 766a3b4..edec94a 100644 --- a/owlps-positioning/TODO +++ b/owlps-positioning/TODO @@ -35,9 +35,6 @@ ° Différencier le pas pour X, Y et Z ? ° Régler le start & stop dans MultilaterationAlgorithm. -- Mobile - Attributs Viterbi ? (Cf. l'ancien clientinfo.hh.) - - Renommages de membres ° InputMedium : °° current_line_nb et get_current_line_nb() @@ -45,10 +42,6 @@ ° Input : get_next_request() −> read_next_request() ° Area : p_min et p_max −> coord_min et coord_max -- Doxygen - ° Pourquoi une description détaillée placée dans un .cc ne s'affiche - pas pour operator<<() ? - - « C++ en action » ° Espaces de noms ? 109 ° Réserver l'espace mémoire des vector avec reserve(). 217