Update TODOs
This commit is contained in:
Matteo Cypriani
parent
507072bd24
commit
3ac6b440e4
17
TODO
17
TODO
|
|
@ -6,8 +6,9 @@
|
||||||
° Translate comments & help.
|
° Translate comments & help.
|
||||||
- Use string for network exchanges?
|
- Use string for network exchanges?
|
||||||
- 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 (mostly) done in the owlps-positioning C++ code, but not
|
||||||
in C modules. [Done in libowlps & libowlps-client.]
|
constantly in C modules.
|
||||||
|
[Done in libowlps & libowlps-client.]
|
||||||
- Allow to use hostnames instead of IP addresses in all modules
|
- Allow to use hostnames instead of IP addresses in all modules
|
||||||
getaddrinfo(3) (or gethostbyname(3)) should be used when opening
|
getaddrinfo(3) (or gethostbyname(3)) should be used when opening
|
||||||
sockets in libowlps, then string buffer size have to be adapted in
|
sockets in libowlps, then string buffer size have to be adapted in
|
||||||
|
|
@ -29,18 +30,18 @@
|
||||||
|
|
||||||
* Aggregator
|
* 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
|
- 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!
|
||||||
|
- inet_ntoa() is not secure with threads
|
||||||
|
(But it is currently used by only one thread.) Use inet_ntop()
|
||||||
|
instead?
|
||||||
- Refactor got_request().
|
- Refactor got_request().
|
||||||
- Use the type of a request to identify it?
|
- Use the type of a request to identify it?
|
||||||
(along with the mobile MAC address and the request time)
|
(along with the mobile MAC address and the request time)
|
||||||
- got_request(): option for the maximal difference time
|
- got_request(): option for the maximal difference time
|
||||||
For implicit packet, we consider that packet from the same MAC and
|
For implicit packets, we consider that packets from the same MAC
|
||||||
received within an interval of 10ms are part of the same
|
and received within an interval of 10ms are part of the same
|
||||||
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.
|
||||||
|
|
@ -54,7 +55,7 @@
|
||||||
Currently the size is corrected only for data packets.
|
Currently the size is corrected only for data packets.
|
||||||
- Move endianess #defines in libowlps?
|
- Move endianess #defines in libowlps?
|
||||||
- Merge Makefile and Makefile_atheros?
|
- Merge Makefile and Makefile_atheros?
|
||||||
Use autohell?
|
Use autohell, cmake, etc.?
|
||||||
- Use positive (USE_CONFUSE) and negative (NO_USE_PTHREAD) options?
|
- Use positive (USE_CONFUSE) and negative (NO_USE_PTHREAD) options?
|
||||||
The default options could be modified, but that's not so important,
|
The default options could be modified, but that's not so important,
|
||||||
is it?
|
is it?
|
||||||
|
|
|
||||||
|
|
@ -7,6 +7,8 @@
|
||||||
streams (no need to cast each time anymore).
|
streams (no need to cast each time anymore).
|
||||||
|
|
||||||
- User interface
|
- User interface
|
||||||
|
° Verbose and/or debug level (should replace #ifdef DEBUG): print
|
||||||
|
the options, etc.
|
||||||
° Case-insensitive string comparison (for algorithm names, etc.).
|
° Case-insensitive string comparison (for algorithm names, etc.).
|
||||||
° Use a prefix for configuration files (search for config files set
|
° Use a prefix for configuration files (search for config files set
|
||||||
with relative path in owlps-positioning.cfg in the same directory).
|
with relative path in owlps-positioning.cfg in the same directory).
|
||||||
|
|
@ -38,12 +40,12 @@
|
||||||
- MinMax
|
- MinMax
|
||||||
° Différencier le pas pour X, Y et Z ?
|
° Différencier le pas pour X, Y et Z ?
|
||||||
|
|
||||||
- Renommages de membres
|
- Members renaming
|
||||||
° InputMedium :
|
° InputMedium:
|
||||||
°° current_line_nb et get_current_line_nb()
|
°° current_line_nb & get_current_line_nb()
|
||||||
°° get_next_request() −> read_next_request()
|
°° get_next_request() −> read_next_request()
|
||||||
° 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
|
||||||
|
|
||||||
- « C++ en action »
|
- « C++ en action »
|
||||||
° Espaces de noms ? 109
|
° Espaces de noms ? 109
|
||||||
|
|
@ -59,8 +61,8 @@
|
||||||
respecter ce principe en utilisant mieux les opérateurs et en ne
|
respecter ce principe en utilisant mieux les opérateurs et en ne
|
||||||
proposant pas d'accesseur direct.
|
proposant pas d'accesseur direct.
|
||||||
|
|
||||||
- Divers
|
- Miscellaneous
|
||||||
° Passage de pointeurs en argument : const * const <type>
|
° Pointer function arguments: const * const <type>
|
||||||
|
|
||||||
- Timestamp
|
- Timestamp
|
||||||
Dans les tests, il y a une probabilité de 1/1000000 que la valeur
|
Dans les tests, il y a une probabilité de 1/1000000 que la valeur
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue