decoder: rework how to swap frame layout.

[apps/agl-service-can-low-level.git] / low-can-binding / utils / converter.cpp

diff --git a/low-can-binding/utils/converter.cpp b/low-can-binding/utils/converter.cpp
index 9e70bc9..6392fbd 100644 (file)
--- a/low-can-binding/utils/converter.cpp
+++ b/low-can-binding/utils/converter.cpp
@@ -32,7 +32,7 @@ std::string converter_t::to_hex(const uint8_t data[], const size_t length)
        std::stringstream stream;
        stream << std::hex << std::setfill('0');
        for(int i = 0; i < length; i++)
-               stream << std::hex << ((int) data[i]);
+               stream << std::setfill('0') << std::setw(2) << std::hex << ((int) data[i]);
 
        return stream.str();
 }
@@ -48,7 +48,7 @@ std::string converter_t::to_hex(const uint8_t data[], const size_t length)
  * @param new_start_bit The first bit of the signal in the frame
  * @param new_end_bit The last bit of the signal in the frame
  */
-void converter_t::signal_to_bits_bytes(uint32_t bit_position, uint32_t bit_size, int &new_start_byte, int &new_end_byte, uint8_t &new_start_bit, uint8_t &new_end_bit)
+void converter_t::signal_to_bits_bytes(unsigned int bit_position, unsigned int bit_size, int &new_start_byte, int &new_end_byte, uint8_t &new_start_bit, uint8_t &new_end_bit)
 {
        new_start_byte = bit_position >> 3;
        new_start_bit = bit_position % 8;
@@ -58,27 +58,31 @@ void converter_t::signal_to_bits_bytes(uint32_t bit_position, uint32_t bit_size,
 
 
 /**
- * @brief      This is to use when you have a big endian CAN frame layout.
- *                     It converts the bit position so it matches with little endiant CAN frame layout.
+ * @brief      This is to use when you have a big endian CAN frame layout.
+ *             It converts the bit position so it matches with little endiant CAN frame layout.
  *
  * @param bit_position         Original bit position.
  * @param bit_size             Size of the data.
- * @return uint32_t    New little endian bit position.
+ * @return uint32_t    New bit position.
  */
-uint32_t converter_t::bit_position_swap(uint32_t bit_position,uint32_t bit_size)
+uint32_t converter_t::bit_position_swap(unsigned int msg_length, unsigned int bit_position, unsigned int bit_size)
 {
-       uint32_t start_byte_position = (uint32_t)(bit_position/8);
-       uint32_t bit_size_rest = bit_size;
+       return msg_length - bit_position - bit_size;
+       /*
+       unsigned int start_byte_position = (unsigned int)(bit_position/8);
+       unsigned int bit_size_rest = bit_size;
 
-       if((int)(bit_size-(8 + start_byte_position*8-bit_position%8))>0)
+       if((int)(bit_size-(8 + start_byte_position * 8 - bit_position % 8)) > 0)
        {
                AFB_ERROR("Error: bit_position and bit_size getting out of range");
                return bit_position;
        }
 
-       if(bit_size<=8 && ((bit_position+bit_size)%8==bit_size || (bit_position+bit_size)%8==0))
+       if(bit_size <= 8 &&
+          ((bit_position+bit_size) % 8 == bit_size ||
+          (bit_position+bit_size)%8==0))
        {
-               return (uint32_t)(start_byte_position*8 + (8-bit_size));
+               return (unsigned int)(start_byte_position*8 + (8-bit_size));
        }
        else
        {
@@ -88,7 +92,7 @@ uint32_t converter_t::bit_position_swap(uint32_t bit_position,uint32_t bit_size)
                        start_byte_position--;
                        bit_position = start_byte_position*8;
                } while (bit_size_rest>8);
-               return (uint32_t)(start_byte_position*8 + (8-bit_size_rest));
+               return (unsigned int)(start_byte_position*8 + (8-bit_size_rest));
        }
-
+       */
 }