Code Review
/
apps
/
agl-service-can-low-level.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Code format and style changes
[apps/agl-service-can-low-level.git]
/
low-can-binding
/
can
/
can-bus.cpp
diff --git
a/low-can-binding/can/can-bus.cpp
b/low-can-binding/can/can-bus.cpp
index
d5ad485
..
49a6b60
100644
(file)
--- a/
low-can-binding/can/can-bus.cpp
+++ b/
low-can-binding/can/can-bus.cpp
@@
-33,6
+33,16
@@
#include "../utils/signals.hpp"
#include "../utils/openxc-utils.hpp"
#include "../utils/signals.hpp"
#include "../utils/openxc-utils.hpp"
+/// @brief Class destructor
+///
+/// @param[in] conf_file - Stop threads and unlock them to correctly finish them
+/// even without any activity on the CAN bus.
+can_bus_t::~can_bus_t()
+{
+ stop_threads();
+ new_can_message_cv_.notify_one();
+}
+
/// @brief Class constructor
///
/// @param[in] conf_file - handle to the json configuration file.
/// @brief Class constructor
///
/// @param[in] conf_file - handle to the json configuration file.
@@
-40,10
+50,10
@@
can_bus_t::can_bus_t(utils::config_parser_t conf_file)
: conf_file_{conf_file}
{}
: conf_file_{conf_file}
{}
-/// @brief Take a decoded message to determine if its value compl
y with the want
ed
-/// filter
ing value
s.
+/// @brief Take a decoded message to determine if its value compl
ies with the desir
ed
+/// filters.
///
///
-/// @param[in] vehicle_message -
A decoded message to analyze
+/// @param[in] vehicle_message -
The decoded message to be analyzed.
/// @param[in] can_subscription - the subscription which will be notified depending
/// on its filtering values. Filtering values are stored in the event_filtermember.
///
/// @param[in] can_subscription - the subscription which will be notified depending
/// on its filtering values. Filtering values are stored in the event_filtermember.
///
@@
-53,10
+63,10
@@
bool can_bus_t::apply_filter(const openxc_VehicleMessage& vehicle_message, std::
bool send = false;
if(is_valid(vehicle_message))
{
bool send = false;
if(is_valid(vehicle_message))
{
- float min =
std::isnan(can_subscription->get_min()) ? -INFINITY :
can_subscription->get_min();
- float max =
std::isnan(can_subscription->get_max()) ? INFINITY :
can_subscription->get_max();
+ float min = can_subscription->get_min();
+ float max = can_subscription->get_max();
double value = get_numerical_from_DynamicField(vehicle_message);
double value = get_numerical_from_DynamicField(vehicle_message);
- send = (value < min
&&
value > max) ? false : true;
+ send = (value < min
||
value > max) ? false : true;
}
return send;
}
}
return send;
}
@@
-91,7
+101,7
@@
void can_bus_t::process_can_signals(const can_message_t& can_message, std::map<i
{
std::lock_guard<std::mutex> decoded_can_message_lock(decoded_can_message_mutex_);
push_new_vehicle_message(subscription_id, vehicle_message);
{
std::lock_guard<std::mutex> decoded_can_message_lock(decoded_can_message_mutex_);
push_new_vehicle_message(subscription_id, vehicle_message);
- AFB_DEBUG("%s CAN signals processed.",
sig->get_name().c_str());
+ AFB_DEBUG("%s CAN signals processed.", sig->get_name().c_str());
}
}
}
}
}
}
@@
-158,8
+168,8
@@
void can_bus_t::can_decode_message()
}
can_message_lock.lock();
}
}
can_message_lock.lock();
}
- new_decoded_can_message_.notify_one();
- can_message_lock.unlock();
+
new_decoded_can_message_.notify_one();
+
can_message_lock.unlock();
}
}
}
}