Move file parsing library to fly::parser namespace

Author: trflynn89

bench/json/benchmark_json.cpp

@@ -36,7 +36,7 @@ class LibflyJsonParser : public JsonParserBase
     }
 
 private:
-    fly::JsonParser m_parser;
+    fly::parser::JsonParser m_parser;
 };
 
 // Boost.JSON - https://github.com/boostorg/json

fly/config/config_manager.cpp

@@ -49,11 +49,11 @@ ConfigManager::ConfigManager(
     switch (file_type)
     {
         case ConfigFileType::Ini:
-            m_parser = std::make_unique<fly::IniParser>();
+            m_parser = std::make_unique<fly::parser::IniParser>();
             break;
 
         case ConfigFileType::Json:
-            m_parser = std::make_unique<fly::JsonParser>();
+            m_parser = std::make_unique<fly::parser::JsonParser>();
             break;
 
         default:

fly/config/config_manager.hpp

@@ -12,9 +12,9 @@
 #include <string>
 #include <type_traits>
 
-namespace fly {
+namespace fly::parser {
 class Parser;
-} // namespace fly
+} // namespace fly::parser
 
 namespace fly::path {
 class PathMonitor;
@@ -113,7 +113,7 @@ class ConfigManager : public std::enable_shared_from_this<ConfigManager>
     std::shared_ptr<fly::task::SequencedTaskRunner> m_task_runner;
 
     std::shared_ptr<fly::path::PathMonitor> m_monitor;
-    std::unique_ptr<fly::Parser> m_parser;
+    std::unique_ptr<fly::parser::Parser> m_parser;
     fly::Json m_values;
 
     const std::filesystem::path m_path;

fly/parser/ini_parser.cpp

@@ -5,22 +5,22 @@
 
 #include <vector>
 
-namespace fly {
+namespace fly::parser {
 
 #define ILOG(...)                                                                                  \
     LOGW("[line {}]: " FLY_FORMAT_STRING(__VA_ARGS__), line() FLY_FORMAT_ARGS(__VA_ARGS__));
 
 //==================================================================================================
-std::optional<Json> IniParser::parse_internal()
+std::optional<fly::Json> IniParser::parse_internal()
 {
-    Json values = JsonTraits::object_type();
-    Json::iterator current;
+    fly::Json values = JsonTraits::object_type();
+    fly::Json::iterator current;
 
     std::string data;
 
     while (getline(data))
     {
-        String::trim(data);
+        fly::String::trim(data);
 
         if (data.empty() || data.starts_with(';'))
         {
@@ -84,7 +84,7 @@ std::optional<Json> IniParser::parse_internal()
         }
     }
 
-    return values.empty() ? std::nullopt : std::optional<Json>(std::move(values));
+    return values.empty() ? std::nullopt : std::optional<fly::Json>(std::move(values));
 }
 
 //==================================================================================================
@@ -106,7 +106,7 @@ bool IniParser::getline(std::string &result)
 //==================================================================================================
 std::optional<std::string> IniParser::on_section(std::string &section)
 {
-    String::trim(section);
+    fly::String::trim(section);
 
     if ((trim_value(section, '\'') != TrimResult::Untrimmed) ||
         (trim_value(section, '\"') != TrimResult::Untrimmed))
@@ -124,14 +124,14 @@ IniParser::on_name_value_pair(const std::string &name_value)
 {
     static constexpr std::uint32_t s_size = 2;
 
-    std::vector<std::string> pair = String::split(name_value, '=', s_size);
+    std::vector<std::string> pair = fly::String::split(name_value, '=', s_size);
 
     if (pair.size() == s_size)
     {
         std::string name(std::move(pair[0])), value(std::move(pair[1]));
 
-        String::trim(name);
-        String::trim(value);
+        fly::String::trim(name);
+        fly::String::trim(value);
 
         if ((trim_value(name, '\'') != TrimResult::Untrimmed) ||
             (trim_value(name, '\"') != TrimResult::Untrimmed))
@@ -181,4 +181,4 @@ IniParser::TrimResult IniParser::trim_value(std::string &str, char start, char e
     return TrimResult::Untrimmed;
 }
 
-} // namespace fly
+} // namespace fly::parser

fly/parser/ini_parser.hpp

@@ -7,7 +7,7 @@
 #include <optional>
 #include <string>
 
-namespace fly {
+namespace fly::parser {
 
 /**
  * Implementation of the Parser interface for the .ini format.
@@ -23,7 +23,7 @@ class IniParser : public Parser
      *
      * @return If successful, the parsed values. Otherwise, an uninitialized value.
      */
-    std::optional<Json> parse_internal() override;
+    std::optional<fly::Json> parse_internal() override;
 
 private:
     /**
@@ -94,4 +94,4 @@ class IniParser : public Parser
     TrimResult trim_value(std::string &str, char start, char end) const;
 };
 
-} // namespace fly
+} // namespace fly::parser