X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fcan-signals.cpp;h=0a6a829ca069b0466ca402d7606017b215ea689b;hb=035493b81d5bb6d909de05e9a06a0f61588a35aa;hp=610a5a8d137b6962c839d986a1cf43e237971739;hpb=6f6364f74a4c454d096b0525bdb7563783ee80b7;p=apps%2Fagl-service-can-low-level.git

diff --git a/src/can-signals.cpp b/src/can-signals.cpp
index 610a5a8d..0a6a829c 100644
--- a/src/can-signals.cpp
+++ b/src/can-signals.cpp
@@ -15,60 +15,68 @@
  * limitations under the License.
  */
 
-#include <string>
-#include <vector>
+#include "can-signals.hpp"
+
 #include <fnmatch.h>
-#include "can-signals.h"
 
-/* Can signal event map making access to afb_event
- * external to openxc existing structure.
+#include "low-can-binding.hpp"
+
+/**
+ * @brief Dumb SIGNALS array. It is composed by CanMessageSet
+ * SIGNALS[MESSAGE_SET_ID][CanSignal]
  */
-std::map <CanSignal, struct afb_event> subscribed_signals;
-std::map <CanSignal, struct afb_event>::iterator subscribed_signals_i;
+std::vector<std::vector<CanSignal>> SIGNALS {
+	{}// message set: example
+};
 
-/* Find one or many signals based on its name or id
-* passed through openxc_DynamicField.
-*/
-#define MESSAGE_SET_ID 0
-std::vector <CanSignal> find_can_signals(openxc_DynamicField &key)
+const std::vector<CanSignal> getSignals()
 {
-    std::vector <CanSignal> signals;
-	int n_signals, i;
+	return SIGNALS[MESSAGE_SET_ID];
+}
 
-	n_signals = getSignalCount();
+size_t getSignalCount()
+{
+	return SIGNALS[MESSAGE_SET_ID].size();
+}
 
-	switch(key->type):
+std::vector<CanSignal> find_can_signals(openxc_DynamicField& key)
+{
+	std::vector<CanSignal> signals;
+
+	switch(key.type)
 	{
 		case openxc_DynamicField_Type::openxc_DynamicField_Type_STRING:
-			for(const CanSignal& s : SIGNALS[MESSAGE_SET_ID])
+			for(const CanSignal& s : getSignals())
 			{
-				if(fnmatch(key->string_value, s.genericName) == 0)
+				if(fnmatch(key.string_value, s.genericName, FNM_CASEFOLD) == 0)
 					signals.push_back(s);
 			}
 			break;
 		case openxc_DynamicField_Type::openxc_DynamicField_Type_NUM:
-			for(const CanSignal& s : SIGNALS[MESSAGE_SET_ID])
+			for(const CanSignal& s : getSignals())
 			{
 				CanMessageDefinition *msg_def = s.message;
-				if(msg_def->id == key->numeric_value)
-					signals.push_back(s)
+				if(msg_def->id == key.numeric_value)
+					signals.push_back(s);
 			}
 			break;
 		default:
-			ERROR(interface, "find_can_signals: wrong openxc_DynamicField specified. Use openxc_DynamicField_Type_NUM or openxc_DynamicField_Type_STRING type only.");
-			return NULL;
+			ERROR(binder_interface, "find_can_signals: wrong openxc_DynamicField specified. Use openxc_DynamicField_Type_NUM or openxc_DynamicField_Type_STRING type only.");
+			CanSignal cs;
+			::memset(&cs, 0, sizeof(CanSignal));
+			signals.push_back(cs);
+			return signals;
 			break;
 	}
-
 	return signals;
 }
 
-std::vector<CanSignal>& getSignals()
+inline uint32_t get_CanSignal_id(const CanSignal& sig)
 {
-	return SIGNALS[MESSAGE_SET_ID];
+	return sig.message->id;
 }
 
-int getSignalCount()
+const std::map<std::string, struct afb_event> get_subscribed_signals()
 {
-	return SIGNALS[MESSAGE_SET_ID].size();
+	return subscribed_signals;
 }
\ No newline at end of file