diff options
author | Romain Forlot <romain.forlot@iot.bzh> | 2017-05-19 00:36:30 +0200 |
---|---|---|
committer | Romain Forlot <romain.forlot@iot.bzh> | 2017-05-19 14:48:23 +0200 |
commit | 53e09a3a8e9ff405ee5936f4765659d3817991e6 (patch) | |
tree | 0236e6f5e20e9622e4c30cc5d00fdd1cf6f2c2b9 | |
parent | 3bc17dd60bfd97e36b12849d4d3a5dd3d093a426 (diff) |
Change method name to signify that it is an aggregate.
As can_message_definitions got the exact same name but the one from can_message_set
is an aggregate of all can_message_definition. So it is clearer using a different
name.
Change-Id: Ieb9e4d3291f3dc460eb352d8b8fe5a7cf479e687
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
4 files changed, 6 insertions, 6 deletions
diff --git a/CAN-binder/low-can-binding/binding/configuration.cpp b/CAN-binder/low-can-binding/binding/configuration.cpp index cc2b8f23..4898bdcb 100644 --- a/CAN-binder/low-can-binding/binding/configuration.cpp +++ b/CAN-binder/low-can-binding/binding/configuration.cpp @@ -47,9 +47,9 @@ const std::vector<std::shared_ptr<can_message_set_t> >& configuration_t::get_can return can_message_set_; } -std::vector<std::shared_ptr<can_signal_t> > configuration_t::get_can_signals() +std::vector<std::shared_ptr<can_signal_t> > configuration_t::get_all_can_signals() { - return can_message_set_[active_message_set_]->get_can_signals(); + return can_message_set_[active_message_set_]->get_all_can_signals(); } std::vector<std::shared_ptr<diagnostic_message_t> > configuration_t::get_diagnostic_messages() diff --git a/CAN-binder/low-can-binding/binding/configuration.hpp b/CAN-binder/low-can-binding/binding/configuration.hpp index a3bb1894..fd669237 100644 --- a/CAN-binder/low-can-binding/binding/configuration.hpp +++ b/CAN-binder/low-can-binding/binding/configuration.hpp @@ -67,7 +67,7 @@ class configuration_t const std::vector<std::shared_ptr<can_message_set_t> >& get_can_message_set(); - std::vector<std::shared_ptr<can_signal_t> > get_can_signals(); + std::vector<std::shared_ptr<can_signal_t> > get_all_can_signals(); std::vector<std::shared_ptr<diagnostic_message_t> > get_diagnostic_messages(); diff --git a/CAN-binder/low-can-binding/can/can-bus.cpp b/CAN-binder/low-can-binding/can/can-bus.cpp index bd9cde77..c2968054 100644 --- a/CAN-binder/low-can-binding/can/can-bus.cpp +++ b/CAN-binder/low-can-binding/can/can-bus.cpp @@ -77,7 +77,7 @@ int can_bus_t::process_can_signals(const can_message_t& can_message) if( s.find(sig->get_name()) != s.end() && afb_event_is_valid(s[sig->get_name()])) { bool send = true; - decoded_message = decoder_t::translateSignal(*sig, can_message, conf.get_can_signals(), &send); + decoded_message = decoder_t::translateSignal(*sig, can_message, conf.get_all_can_signals(), &send); if(send) { diff --git a/CAN-binder/low-can-binding/utils/signals.cpp b/CAN-binder/low-can-binding/utils/signals.cpp index b75eb2c5..261f572b 100644 --- a/CAN-binder/low-can-binding/utils/signals.cpp +++ b/CAN-binder/low-can-binding/utils/signals.cpp @@ -60,11 +60,11 @@ namespace utils switch(key.type) { case openxc_DynamicField_Type::openxc_DynamicField_Type_STRING: - lookup_signals_by_name(key.string_value, configuration_t::instance().get_can_signals(), sf.can_signals); + lookup_signals_by_name(key.string_value, configuration_t::instance().get_all_can_signals(), sf.can_signals); lookup_signals_by_name(key.string_value, configuration_t::instance().get_diagnostic_messages(), sf.diagnostic_messages); break; case openxc_DynamicField_Type::openxc_DynamicField_Type_NUM: - lookup_signals_by_id(key.numeric_value, configuration_t::instance().get_can_signals(), sf.can_signals); + lookup_signals_by_id(key.numeric_value, configuration_t::instance().get_all_can_signals(), sf.can_signals); lookup_signals_by_id(key.numeric_value, configuration_t::instance().get_diagnostic_messages(), sf.diagnostic_messages); break; default: |