From d3ec67b912df6e67239cd5b283135d077dc78744 Mon Sep 17 00:00:00 2001 From: Romain Forlot Date: Sat, 23 Nov 2019 19:52:30 +0100 Subject: [PATCH] all: format Remove unecessary brackets and replace if possible if...else by ternary operator. Change-Id: Ibe46b83dea976e050b9cd9eb7ee7ab2c33b63905 Signed-off-by: Romain Forlot --- low-can-binding/binding/application.cpp | 4 -- low-can-binding/binding/low-can-subscription.cpp | 13 ++----- low-can-binding/can/can-decoder.cpp | 6 --- low-can-binding/can/message/can-message.cpp | 8 ++-- low-can-binding/can/message/j1939-message.cpp | 49 +++++++----------------- 5 files changed, 20 insertions(+), 60 deletions(-) diff --git a/low-can-binding/binding/application.cpp b/low-can-binding/binding/application.cpp index bb3bf15b..0b8963cb 100644 --- a/low-can-binding/binding/application.cpp +++ b/low-can-binding/binding/application.cpp @@ -171,13 +171,9 @@ bool application_t::is_engine_on() if(sf.diagnostic_messages.front()->get_last_value_with_timestamp().first > 0 && std::difftime(std::time(nullptr), last_timestamp_in_s) < ENGINE_VALUE_TIMEOUT) - { engine_on = true; - } else - { AFB_NOTICE("is_engine_on: engine.speed diagnostic message found, but engine seems off"); - } } else { diff --git a/low-can-binding/binding/low-can-subscription.cpp b/low-can-binding/binding/low-can-subscription.cpp index aef7ebfe..0f1d1da7 100644 --- a/low-can-binding/binding/low-can-subscription.cpp +++ b/low-can-binding/binding/low-can-subscription.cpp @@ -648,7 +648,6 @@ int low_can_subscription_t::create_rx_filter(std::shared_ptrwrite_message(msg); @@ -714,8 +711,6 @@ int low_can_subscription_t::tx_send(low_can_subscription_t &subscription, messag struct bcm_msg &bcm_cm = cm->get_bcm_msg(); - - if(cfd_vect.size() > 1) { AFB_ERROR("Multi frame BCM not implemented"); @@ -738,9 +733,7 @@ int low_can_subscription_t::tx_send(low_can_subscription_t &subscription, messag } if(! subscription.socket_.get()) - { - return -1; - } + return -1; return 0; } diff --git a/low-can-binding/can/can-decoder.cpp b/low-can-binding/can/can-decoder.cpp index 7f92920f..f38840e4 100644 --- a/low-can-binding/can/can-decoder.cpp +++ b/low-can-binding/can/can-decoder.cpp @@ -111,12 +111,6 @@ float decoder_t::parse_signal_bitfield(signal_t& signal, std::shared_ptr 255) -// AFB_ERROR("Error signal %s to long bit size", signal.get_name().c_str()); - -// if(new_start_bit > 255) -// AFB_ERROR("Too long signal offset %d", new_start_bit); - if(data_signal.size() > 65535) AFB_ERROR("Too long data signal %s", signal.get_name().c_str()); diff --git a/low-can-binding/can/message/can-message.cpp b/low-can-binding/can/message/can-message.cpp index 5c3e0f0e..a8843b9f 100644 --- a/low-can-binding/can/message/can-message.cpp +++ b/low-can-binding/can/message/can-message.cpp @@ -131,9 +131,7 @@ std::shared_ptr can_message_t::convert_from_frame(const struct ca if(rtr_flag) length = frame.len& 0xF; else - { length = (frame.len > maxdlen) ? maxdlen : frame.len; - } } } else @@ -178,7 +176,9 @@ struct canfd_frame can_message_t::convert_to_canfd_frame() ::memcpy(frame.data, get_data(), length_); } else + { AFB_ERROR("can_message_t not correctly initialized to be sent"); + } return frame; } @@ -261,7 +261,9 @@ struct can_frame can_message_t::convert_to_can_frame() ::memcpy(frame.data, get_data(), length_); } else + { AFB_ERROR("can_message_t not correctly initialized to be sent"); + } return frame; } @@ -276,9 +278,7 @@ std::string can_message_t::get_debug_message() std::string ret = ""; ret = ret + "Here is the next can message : id " + std::to_string(id_) + " length " + std::to_string(length_) + ", data "; for (size_t i = 0; i < data_.size(); i++) - { ret = ret + std::to_string(data_[i]); - } return ret; } diff --git a/low-can-binding/can/message/j1939-message.cpp b/low-can-binding/can/message/j1939-message.cpp index 0dff71f8..743898ff 100644 --- a/low-can-binding/can/message/j1939-message.cpp +++ b/low-can-binding/can/message/j1939-message.cpp @@ -92,13 +92,12 @@ uint8_t j1939_message_t::get_addr() const{ */ std::string to_hex( 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]); - } - return stream.str(); + std::stringstream stream; + stream << std::hex << std::setfill('0'); + for(int i = 0; i < length; i++) + stream << std::hex << ((int) data[i]); + + return stream.str(); } /// @brief Take a sockaddr_can struct and array of data to initialize class members @@ -132,12 +131,10 @@ std::shared_ptr j1939_message_t::convert_from_addr(struct socka data_string = converter_t::to_hex(data, length); for(i=0;i(j1939_message_t(length, data_vector, timestamp, addr.can_addr.j1939.name, addr.can_addr.j1939.pgn, addr.can_addr.j1939.addr)); } @@ -214,30 +211,10 @@ void j1939_message_t::set_sockname(pgn_t pgn, name_t name, uint8_t addr) sockname_.can_family = AF_CAN; sockname_.can_ifindex = 0; - if(addr <= 0 || addr >= UINT8_MAX ) - { - sockname_.can_addr.j1939.addr = J1939_NO_ADDR; - } - else - { - sockname_.can_addr.j1939.addr = addr; - } - - if(name <= 0 || name >= UINT64_MAX ) - { - sockname_.can_addr.j1939.name = J1939_NO_NAME; - } - else - { - sockname_.can_addr.j1939.name = name; - } - - if(pgn <= 0 || pgn > J1939_PGN_MAX) - { - sockname_.can_addr.j1939.pgn = J1939_NO_PGN; - } - else - { - sockname_.can_addr.j1939.pgn = pgn; - } + sockname_.can_addr.j1939.addr = addr <= 0 || addr >= UINT8_MAX ? + J1939_NO_ADDR : addr; + sockname_.can_addr.j1939.name = name <= 0 || name >= UINT64_MAX ? + J1939_NO_NAME : name; + sockname_.can_addr.j1939.pgn = pgn <= 0 || pgn > J1939_PGN_MAX ? + J1939_NO_PGN : pgn; } -- 2.16.6