From: Romain Forlot <romain.forlot@iot.bzh>
Date: Wed, 15 Mar 2017 22:31:23 +0000 (+0100)
Subject: Small fixes
X-Git-Tag: 3.99.1~258
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=6e4b2f9c0bcdfb3106d9f3b5bdb9316fcdebd9b2;p=apps%2Flow-level-can-service.git

Small fixes

Align frequency type to float and removed useless code.

Change-Id: I13d9407808c42ece0d3fd761351752ad661e01e0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
---

diff --git a/src/can/can-bus.cpp b/src/can/can-bus.cpp
index c08d1e8..31cb4b0 100644
--- a/src/can/can-bus.cpp
+++ b/src/can/can-bus.cpp
@@ -73,7 +73,6 @@ int can_bus_t::process_can_signals(can_message_t& can_message)
 
 	/* First we have to found which can_signal_t it is */
 	search_key = build_DynamicField((double)can_message.get_id());
-	signals.clear();
 	configuration_t::instance().find_can_signals(search_key, signals);
 
 	/* Decoding the message ! Don't kill the messenger ! */
diff --git a/src/diagnostic/diagnostic-message.hpp b/src/diagnostic/diagnostic-message.hpp
index 83c4dcc..4feceb4 100644
--- a/src/diagnostic/diagnostic-message.hpp
+++ b/src/diagnostic/diagnostic-message.hpp
@@ -22,6 +22,7 @@
 #include "uds/uds.h"
 #include "../can/can-bus.hpp"
 #include "../can/can-message.hpp"
+#include "active-diagnostic-request.hpp"
 
 #include "../low-can-binding.hpp"
 
@@ -49,7 +50,7 @@ class diagnostic_message_t {
 		int min_; /*!< min_ - Minimum value that can take this pid */
 		int max_; /*!< max_ - Maximum value that can take this pid */
 		enum UNIT unit_; /*!< unit_ : Which unit system is used by that pid. See enum UNIT above.*/
-		int frequency_; /*!< frequency - The frequency to request this PID if supported by the vehicle when automatic, recurring OBD-II requests are enabled.*/
+		float frequency_; /*!< frequency - The frequency to request this PID if supported by the vehicle when automatic, recurring OBD-II requests are enabled.*/
 		DiagnosticResponseDecoder decoder_; /*!< decoder_ - An optional DiagnosticResponseDecoder to parse the payload of responses
 											 * to this request. If the decoder is NULL, the output will include the raw payload
 											 * instead of a parsed value.*/
@@ -66,7 +67,7 @@ class diagnostic_message_t {
 		uint32_t get_pid();
 		const std::string& get_generic_name() const;
 		const std::string get_name() const;
-		int get_frequency() const;
+		float get_frequency() const;
 		DiagnosticResponseDecoder get_decoder() const;
 		DiagnosticResponseCallback get_callback() const;
 
@@ -74,5 +75,4 @@ class diagnostic_message_t {
 
 		bool is_obd2_response(const can_message_t& can_message);
 		bool is_obd2_request(const DiagnosticRequest *request);
-
 };