Use raw pointer to point on parent object.

[apps/low-level-can-service.git] / CAN-binder / low-can-binding / can / can-signals.hpp

diff --git a/CAN-binder/low-can-binding/can/can-signals.hpp b/CAN-binder/low-can-binding/can/can-signals.hpp
index f21cab4..79f56db 100644 (file)
--- a/CAN-binder/low-can-binding/can/can-signals.hpp
+++ b/CAN-binder/low-can-binding/can/can-signals.hpp
@@ -19,15 +19,14 @@
 
 #include <map>
 #include <mutex>
-#include <queue>
 #include <vector>
 #include <string>
+#include <memory>
 
 #include "openxc.pb.h"
 #include "../utils/timer.hpp"
 #include "../utils/socketcan-bcm.hpp"
 #include "can-message.hpp"
-#include "can-message-definition.hpp"
 #include "../diagnostic/diagnostic-message.hpp"
 
 extern "C"
@@ -39,7 +38,7 @@ extern "C"
 #define MESSAGE_SET_ID 0
 
 class can_signal_t;
-
+class can_message_definition_t;
 ///
 /// @brief The type signature for a CAN signal decoder.
 ///
@@ -57,7 +56,7 @@ class can_signal_t;
 /// @return a decoded value in an openxc_DynamicField struct.
 ///
 typedef openxc_DynamicField (*SignalDecoder)(can_signal_t& signal,
-               const std::vector<can_signal_t>& signals, float value, bool* send);
+               const std::vector<std::shared_ptr<can_signal_t> > signals, float value, bool* send);
 
 ///
 /// @brief: The type signature for a CAN signal encoder.
@@ -76,9 +75,8 @@ typedef uint64_t (*SignalEncoder)(can_signal_t* signal,
 class can_signal_t
 {
 private:
+       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::uint8_t message_set_id_; ///< message_set_id_ - Index number to the message_set_id container object
-       std::uint8_t message_id_; ///< message_id_ - Index number to the message_definition_t container object
        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.
                                                  * which make easier to sort message when the come in.*/
@@ -112,9 +110,8 @@ private:
                                                *       this value is undefined. */
 
 public:
+       //can_signal_t(const can_signal_t& b);
        can_signal_t(
-               std::uint8_t message_set_id,
-               std::uint8_t message_id,
                std::string generic_name,
                uint8_t bit_position,
                uint8_t bit_size,
@@ -132,7 +129,7 @@ public:
                bool received);
 
        utils::socketcan_bcm_t get_socket() const;
-       const 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;
@@ -154,8 +151,11 @@ public:
        bool get_received() const;
        float get_last_value() const;
 
+       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);
+
        int create_rx_filter();
+       void read_socket();
 };
\ No newline at end of file