From 77a19477d3cb63b5af519853489cc90ead0a111b Mon Sep 17 00:00:00 2001
From: Corentin Le Gall <corentin.legall@iot.bzh>
Date: Mon, 18 Nov 2019 16:48:20 +0100
Subject: decoder: Fix bit_position swapping

-Added an attribute to signals that tells if the bit_position
has been swapped.
-Test if bit_size and bit_position gives an "out of range" data

Bug-AGL: SPEC-3022

Change-Id: I589565ca923ec807da2d4f0db7c4c92fb737b579
Signed-off-by: Corentin Le Gall <corentin.legall@iot.bzh>
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
---
 low-can-binding/can/signals.hpp | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'low-can-binding/can/signals.hpp')

diff --git a/low-can-binding/can/signals.hpp b/low-can-binding/can/signals.hpp
index bf779981..5f143753 100755
--- a/low-can-binding/can/signals.hpp
+++ b/low-can-binding/can/signals.hpp
@@ -111,6 +111,8 @@ private:
 	sign_t sign_; /* !< sign_ - if the data is signed it indicates the encode */
 	int32_t bit_sign_position_; /*!< bit_sign_position_ - The bit that indicates the sign of the signal in its CAN message*/
 	std::string unit_; /* !< unit_ - The unit of the data */
+	bool bit_position_is_swapped_; /* !<bit_position_is_swapped_- True if the signal's bit position has been
+										swapped (refer to converter_t::bit_position_swap()). Default is false.*/
 
 public:
 
@@ -177,10 +179,12 @@ public:
 	sign_t get_sign() const;
 	int32_t get_bit_sign_position() const;
 	const std::string get_unit() const;
+	bool bit_position_is_swapped() const;
 
 	void set_parent(std::shared_ptr<message_definition_t> parent);
 	void set_received(bool r);
 	void set_last_value(float val);
 	void set_timestamp(uint64_t timestamp);
 	void set_bit_position(uint32_t bit_position);
+	void bit_position_is_swapped_reverse();
 };
-- 
cgit 1.2.3-korg