diff options
| author |   Romain Forlot <romain.forlot@iot.bzh> | 2017-05-12 14:46:19 +0200 |
|---|---|---|
| committer | Romain Forlot <romain.forlot@iot.bzh> | 2017-05-19 11:36:42 +0200 |
| commit | 6f872acf6a3a6c3a7a1a1885e5e66bb7231660b2 (patch) | |
| tree | 35d504f76466e3b7d07e2962208fe114b7866a22 /CAN-binder/low-can-binding/can/can-signals.hpp | |
| parent | 22d0b16875926bdb0a39aa44de2bdde57825127c (diff) | |
Use raw pointer to point on parent object.
Shared pointer on value is a wrong way because once they is no more object
refering to a parent then the pointer on it is destoyed. Not a big deal here
but we better have to use raw pointer.
Change-Id: I58deeb9e82c446ea135be87c11f405da54cc5bff
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Diffstat (limited to 'CAN-binder/low-can-binding/can/can-signals.hpp')
| -rw-r--r-- | CAN-binder/low-can-binding/can/can-signals.hpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/CAN-binder/low-can-binding/can/can-signals.hpp b/CAN-binder/low-can-binding/can/can-signals.hpp index 62fbbe7c..79f56dbf 100644 --- a/CAN-binder/low-can-binding/can/can-signals.hpp +++ b/CAN-binder/low-can-binding/can/can-signals.hpp @@ -75,7 +75,7 @@ typedef uint64_t (*SignalEncoder)(can_signal_t* signal, class can_signal_t { private: - std::shared_ptr<can_message_definition_t> parent_; /*!< parent_ - pointer to the parent message definition holding this signal*/ + can_message_definition_t* parent_; /*!< parent_ - pointer to the parent message definition holding this signal*/ utils::socketcan_bcm_t socket_; /*!< socket_ - Specific BCM socket that filter the signal read from CAN device */ std::string generic_name_; /*!< generic_name_ - The name of the signal to be output.*/ static std::string prefix_; /*!< prefix_ - generic_name_ will be prefixed with it. It has to reflect the used protocol. @@ -129,7 +129,7 @@ public: bool received); utils::socketcan_bcm_t get_socket() const; - std::shared_ptr<can_message_definition_t> get_message() const; + can_message_definition_t* get_message() const; const std::string& get_generic_name() const; const std::string get_name() const; const std::string& get_prefix() const; @@ -151,7 +151,7 @@ public: bool get_received() const; float get_last_value() const; - void set_parent(std::shared_ptr<can_message_definition_t> parent); + void set_parent(can_message_definition_t* parent); void set_prefix(std::string val); void set_received(bool r); void set_last_value(float val); |