Static code review fixes.

[apps/low-level-can-service.git] / CAN-binder / low-can-binding / utils / config-parser.hpp

diff --git a/CAN-binder/low-can-binding/utils/config-parser.hpp b/CAN-binder/low-can-binding/utils/config-parser.hpp
index e6bd9d2..3115e9b 100644 (file)
--- a/CAN-binder/low-can-binding/utils/config-parser.hpp
+++ b/CAN-binder/low-can-binding/utils/config-parser.hpp
@@ -17,27 +17,28 @@
 
 #pragma once
 
-#include <fcntl.h>
-#include <unistd.h>
-#include <sys/ioctl.h>
-#include <json-c/json.h>
-
 #include <string>
+#include <vector>
+#include <utility>
+#include "ini-config.hpp"
 
 namespace utils
 {
+       /// @brief A configuration file parser that handle INI configuration
+       /// file format.
        class config_parser_t
        {
        private:
-               int conf_file_; /*!< conf_file_ - file that handle the binding configuration file */
-               std::string config_content_; /*!< config_content_ - String that contains the content of config file */
-               std::vector<std::string> devices_name_; /*!< devices_name - Found devices name after reading configuration file */
+               const std::string filepath_; /*!< filepath_ - Path to the config file*/
+               ini_config config_content_; /*!< config_content_ - Parsed content of INI file.*/
 
-               void parse_devices_name();
        public:
-               config_parser_t(int conf_file);
+               config_parser_t(config_parser_t&&) = default;
+               config_parser_t(const config_parser_t&) = default;
+               explicit config_parser_t(std::string conf_file);
 
-               void read_conf();
-               std::vector<std::string> get_devices_name();
+               const std::string& filepath() const;
+               bool check_conf();
+               const std::vector<std::pair<std::string, std::string> > get_devices_name();
        };
 }