examples: updates

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

diff --git a/low-can-binding/can/can-encoder.cpp b/low-can-binding/can/can-encoder.cpp
index 2d836cf..113428b 100644 (file)
--- a/low-can-binding/can/can-encoder.cpp
+++ b/low-can-binding/can/can-encoder.cpp
@@ -37,37 +37,32 @@ void encoder_t::encode_data(std::shared_ptr<signal_t> sig, std::vector<uint8_t>
        uint32_t bit_position = sig->get_bit_position();
        int new_start_byte = 0;
        int new_end_byte = 0;
-       int new_start_bit_tmp = 0;
-       int new_end_bit = 0;
+       uint8_t new_start_bit = 0;
+       uint8_t new_end_bit = 0;
 
-       converter_t::signal_to_bits_bytes(bit_position, bit_size, new_start_byte, new_end_byte, new_start_bit_tmp, new_end_bit);
+       converter_t::signal_to_bits_bytes(bit_position, bit_size, new_start_byte, new_end_byte, new_start_bit, new_end_bit);
 
        int len_signal_bytes_tmp = new_end_byte - new_start_byte + 1;
 
        uint8_t len_signal_bytes = 0;
        if(len_signal_bytes_tmp > 255)
        {
-               AFB_ERROR("Error signal %s too long",sig->get_name().c_str());
+               AFB_ERROR("Error signal %s too long", sig->get_name().c_str());
        }
        else
        {
                len_signal_bytes = (uint8_t) len_signal_bytes_tmp;
        }
-
-       uint8_t new_start_bit = 0;
-       if(new_start_bit_tmp > 255)
-       {
-               AFB_ERROR("Error signal %s too long",sig->get_name().c_str());
-       }
-       else
+/*
+       if(new_start_bit > 255)
        {
-               new_start_bit = (uint8_t) new_start_bit_tmp;
+               AFB_ERROR("Error signal %s too long", sig->get_name().c_str());
        }
-
+*/
        uint8_t new_bit_size = 0;
        if(bit_size > 255)
        {
-               AFB_ERROR("Error signal %s to long bit size",sig->get_name().c_str());
+               AFB_ERROR("Error signal %s to long bit size", sig->get_name().c_str());
        }
        else
        {
@@ -153,7 +148,7 @@ message_t* encoder_t::build_frame(const std::shared_ptr<signal_t>& signal, uint6
 
        for(const auto& sig: signal->get_message()->get_signals())
        {
-               encode_data(sig,data,false,factor,offset);
+               encode_data(sig, data, false, factor, offset);
        }
        message->set_data(data);
        return message;
@@ -182,7 +177,7 @@ message_t* encoder_t::build_message(const std::shared_ptr<signal_t>& signal, uin
                                                                         data,
                                                                         0);
 
-               return build_frame(signal,value,message, factor, offset);
+               return build_frame(signal, value, message, factor, offset);
        }
 #ifdef USE_FEATURE_J1939
        else if(signal->get_message()->is_j1939())
@@ -193,7 +188,7 @@ message_t* encoder_t::build_message(const std::shared_ptr<signal_t>& signal, uin
                                                                           J1939_NO_NAME,
                                                                           signal->get_message()->get_id(),
                                                                           J1939_NO_ADDR);
-               return build_frame(signal,value,message, factor, offset);
+               return build_frame(signal, value, message, factor, offset);
        }
 #endif
        else