From 53e09a3a8e9ff405ee5936f4765659d3817991e6 Mon Sep 17 00:00:00 2001 From: Romain Forlot Date: Fri, 19 May 2017 00:36:30 +0200 Subject: [PATCH] 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 --- CAN-binder/low-can-binding/binding/configuration.cpp | 4 ++-- CAN-binder/low-can-binding/binding/configuration.hpp | 2 +- CAN-binder/low-can-binding/can/can-bus.cpp | 2 +- CAN-binder/low-can-binding/utils/signals.cpp | 4 ++-- 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 >& configuration_t::get_can return can_message_set_; } -std::vector > configuration_t::get_can_signals() +std::vector > 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 > 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 >& get_can_message_set(); - std::vector > get_can_signals(); + std::vector > get_all_can_signals(); std::vector > 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: -- 2.16.6