diff options
| author |   Romain Forlot <romain.forlot@iot.bzh> | 2017-03-09 10:16:45 +0100 |
|---|---|---|
| committer | Romain Forlot <romain.forlot@iot.bzh> | 2017-03-16 17:10:40 +0100 |
| commit | 3b55101caf70e27cb312f3f7235ed4b30ff6d849 (patch) | |
| tree | 05040a840a32f2791f383c94121557c259f471c9 | |
| parent | 1da3cb296788fa0a32f3669edf4c2539e80ad2e2 (diff) | |
Fix: use of new objects and type conversion missing.
Change-Id: I583ad536de619fed2be655a493299fe43cc65924
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
| -rw-r--r-- | src/utils/signals.cpp | 4 | ||||
| -rw-r--r-- | src/utils/signals.hpp | 8 |
2 files changed, 6 insertions, 6 deletions
diff --git a/src/utils/signals.cpp b/src/utils/signals.cpp index e65d35e..7393409 100644 --- a/src/utils/signals.cpp +++ b/src/utils/signals.cpp @@ -61,11 +61,11 @@ std::vector<std::string> find_signals(const openxc_DynamicField &key) { case openxc_DynamicField_Type::openxc_DynamicField_Type_STRING: lookup_signals_by_name(key.string_value, config->get_can_signals(), found_signals_name); - lookup_signals_by_name(key.string_value, get_obd2_signals(), found_signals_name); + lookup_signals_by_name(key.string_value, config->get_obd2_signals(), found_signals_name); break; case openxc_DynamicField_Type::openxc_DynamicField_Type_NUM: lookup_signals_by_id(key.numeric_value, config->get_can_signals(), found_signals_name); - lookup_signals_by_id(key.numeric_value, get_obd2_signals(), found_signals_name); + lookup_signals_by_id(key.numeric_value, config->get_obd2_signals(), found_signals_name); break; default: ERROR(binder_interface, "find_signals: wrong openxc_DynamicField specified. Use openxc_DynamicField_Type_NUM or openxc_DynamicField_Type_STRING type only."); diff --git a/src/utils/signals.hpp b/src/utils/signals.hpp index db3a401..fe86953 100644 --- a/src/utils/signals.hpp +++ b/src/utils/signals.hpp @@ -52,9 +52,9 @@ void lookup_signals_by_name(const std::string& key, std::vector<T>& signals, std template <typename T> void lookup_signals_by_name(const std::string& key, std::vector<T>& signals, std::vector<std::string>& found_signals_name) { - for(const T& s : signals) + for(T& s : signals) { - if(::fnmatch(key.c_str(), s.generic_name, FNM_CASEFOLD) == 0) + if(::fnmatch(key.c_str(), s.get_generic_name().c_str(), FNM_CASEFOLD) == 0) found_signals_name.push_back(s.get_generic_name()); } } @@ -74,11 +74,11 @@ void lookup_signals_by_id(const double key, std::vector<T>& signals, std::vector template <typename T> void lookup_signals_by_id(const double key, std::vector<T>& signals, std::vector<std::string>& found_signals_name) { - for(const T& s : signals) + for(T& s : signals) { if(config->get_signal_id(s) == key) { - found_signals_name.push_back(s.generic_name); + found_signals_name.push_back(s.get_generic_name()); } } } |