Update TODOs
This commit is contained in:
parent
d1c6abdaf7
commit
e8332fd0f6
21
TODO
21
TODO
|
@ -5,7 +5,10 @@
|
||||||
- Mark arguments as const in function headers if needed
|
- Mark arguments as const in function headers if needed
|
||||||
That is done in the owlps-positioning C++ code, but not constantly
|
That is done in the owlps-positioning C++ code, but not constantly
|
||||||
in C modules. [Done in libowlps & libowlps-client.]
|
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?
|
- Use struct ether_addr to store MAC addresses?
|
||||||
We could use the struct ether_addr to store binary MAC addresses,
|
We could use the struct ether_addr to store binary MAC addresses,
|
||||||
and convert them to strings with ether_ntoa() instead of
|
and convert them to strings with ether_ntoa() instead of
|
||||||
|
@ -14,14 +17,18 @@
|
||||||
|
|
||||||
* libowlps
|
* libowlps
|
||||||
|
|
||||||
- Delete timestamp_is_null()?
|
- Currently unused functions:
|
||||||
This function is currently not used.
|
° 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
|
* Aggregator
|
||||||
|
|
||||||
- inet_ntoa() is not secure with threads
|
- 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
|
- Use locks to read/write the AP list
|
||||||
Currently, several threads can access the list simultaneously, and
|
Currently, several threads can access the list simultaneously, and
|
||||||
that's not cool!
|
that's not cool!
|
||||||
|
@ -34,14 +41,14 @@
|
||||||
request. We should define an option to allow user to choose the
|
request. We should define an option to allow user to choose the
|
||||||
time he wants.
|
time he wants.
|
||||||
- Allow blank parameters for output options.
|
- Allow blank parameters for output options.
|
||||||
- Fix segfault on SIGINT.
|
|
||||||
|
|
||||||
|
|
||||||
* Listener
|
* Listener
|
||||||
|
|
||||||
|
- Refactor and review read_packet().
|
||||||
|
- Listen for autocalibration requests without sending requests?
|
||||||
- read_packet(): use ieee80211_header_size for all implicit packets
|
- read_packet(): use ieee80211_header_size for all implicit packets
|
||||||
Currently the size is corrected only for data 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?
|
- Move endianess #defines in libowlps?
|
||||||
- Merge Makefile and Makefile_atheros?
|
- Merge Makefile and Makefile_atheros?
|
||||||
Use autohell?
|
Use autohell?
|
||||||
|
@ -52,7 +59,7 @@
|
||||||
|
|
||||||
* Client
|
* Client
|
||||||
|
|
||||||
- Allow to use a string for the direction
|
- Allow to use a string for the direction?
|
||||||
Could be nice, but probably useless.
|
Could be nice, but probably useless.
|
||||||
|
|
||||||
|
|
||||||
|
|
|
@ -35,9 +35,6 @@
|
||||||
° Différencier le pas pour X, Y et Z ?
|
° Différencier le pas pour X, Y et Z ?
|
||||||
° Régler le start & stop dans MultilaterationAlgorithm.
|
° Régler le start & stop dans MultilaterationAlgorithm.
|
||||||
|
|
||||||
- Mobile
|
|
||||||
Attributs Viterbi ? (Cf. l'ancien clientinfo.hh.)
|
|
||||||
|
|
||||||
- Renommages de membres
|
- Renommages de membres
|
||||||
° InputMedium :
|
° InputMedium :
|
||||||
°° current_line_nb et get_current_line_nb()
|
°° current_line_nb et get_current_line_nb()
|
||||||
|
@ -45,10 +42,6 @@
|
||||||
° Input : get_next_request() −> read_next_request()
|
° Input : get_next_request() −> read_next_request()
|
||||||
° Area : p_min et p_max −> coord_min et coord_max
|
° 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 »
|
- « C++ en action »
|
||||||
° Espaces de noms ? 109
|
° Espaces de noms ? 109
|
||||||
° Réserver l'espace mémoire des vector avec reserve(). 217
|
° Réserver l'espace mémoire des vector avec reserve(). 217
|
||||||
|
|
Loading…
Reference in New Issue