0eeca3ee0b
AP_RangeFinder: remove case indent
8 years ago
4ca1eefd50
AP_RangeFinder: change magic numbers to define
8 years ago
6254787186
AP_RangeFinder: remove break in LEDDARONE_MODOBUS_INIT case and and use () around (read_len == 0) for clarity
8 years ago
5e5a806979
AP_RangeFinder: modified regarding to advices from @magicrub - san.
8 years ago
4b127edd5e
AP_RangeFinder: fixed LeddarOne busy wait
8 years ago
152edf7189
Global: remove mode line from headers
...
Using a global .dir-locals.el file is a better alternative than
reincluding the same emacs header in every file of the project.
8 years ago
8432c5fb4a
AP_RangeFinder: remove number_detections member variable and add number_detections reference argument to parse_response
8 years ago
43f2482abd
AP_RangeFinder: modify spelling and change C type cast to static_cast
8 years ago
6123fad124
AP_RangeFinder: change error and ok status defines to LeddarOne_Status enum
8 years ago
d58c193c90
AP_RangeFinder: To define the OK status to LEDDARONE status.
8 years ago
3a1cd20f11
AP_RangeFinder: To move to the place that use the variable definition for the first time.
8 years ago
5512b6de02
AP_RangeFinder: Cleanup and added comments
...
- uart->available(); returns uint32 but was stored locally as int32 and treated as uint32. Now stored correctly as uint32
- some variables were set to zero at start of function, then reset to zero before being used. wasted work
9 years ago
e928e20b11
AP_RangeFinder: Ensure no divide by zeros in LeddarOne
...
Having a critical divide by value as a class member that could change external to the function using it is fragile. It was not very obvious that a divide by zero was not possible in the current design, now it's very obvious and safer in case the code changes later.
9 years ago
3123bd6d7d
AP_RangeFinder: add LeddarOne support
9 years ago
Shingo Matsuura
Mathieu OTHACEHE
murata
Tom Pittenger