Message definition can return ref instead of value.
authorRomain Forlot <romain.forlot@iot.bzh>
Fri, 12 May 2017 12:08:16 +0000 (14:08 +0200)
committerRomain Forlot <romain.forlot@iot.bzh>
Fri, 19 May 2017 09:36:42 +0000 (11:36 +0200)
Little optimization that will pass a ref instead of a copy value.

Change-Id: I62fbaaa2fb01cf809e60b592a3d91f06d7235205
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
CAN-binder/low-can-binding/can/can-message-definition.cpp
CAN-binder/low-can-binding/can/can-message-definition.hpp

index e947026..407ed41 100644 (file)
@@ -92,7 +92,7 @@ uint32_t can_message_definition_t::get_id() const
        return id_;
 }
 
-std::vector<std::shared_ptr<can_signal_t> > can_message_definition_t::get_can_signals()
+std::vector<std::shared_ptr<can_signal_t> >& can_message_definition_t::get_can_signals()
 {
        return can_signals_;
 }
index 87c48ac..0e7e5ee 100644 (file)
@@ -60,7 +60,7 @@ public:
 
        const std::string& get_bus_name() const;
        uint32_t get_id() const;
-       std::vector<std::shared_ptr<can_signal_t> > get_can_signals();
+       std::vector<std::shared_ptr<can_signal_t> >& get_can_signals();
 
        void set_parent(std::shared_ptr<can_message_set_t> parent);
        void set_last_value(const can_message_t& cm);