~singpolyma/biboumi

4cc3dc03fc8419abde6e77dceda773a19a246c43 — louiz’ 5 years ago e63f791
The config module doesn’t use the logger at all
M louloulibs/CMakeLists.txt => louloulibs/CMakeLists.txt +0 -1
@@ 99,7 99,6 @@ target_link_libraries(utils ${ICONV_LIBRARIES})
file(GLOB source_config
  config/*.[hc]pp)
add_library(config STATIC ${source_config})
target_link_libraries(config utils)

#
## logger

M louloulibs/config/config.cpp => louloulibs/config/config.cpp +3 -4
@@ 1,8 1,7 @@
#include <config/config.hpp>
#include <logger/logger.hpp>

#include <iostream>
#include <cstring>
#include <sstream>

#include <cstdlib>



@@ 66,7 65,7 @@ bool Config::read_conf(const std::string& name)
  std::ifstream file(Config::filename.data());
  if (!file.is_open())
    {
      log_error("Error while opening file ", filename, " for reading: ", strerror(errno));
      std::cerr << "Error while opening file " << filename << " for reading: " << strerror(errno) << std::endl;
      return false;
    }



@@ 96,7 95,7 @@ void Config::save_to_file()
  std::ofstream file(Config::filename.data());
  if (file.fail())
    {
      log_error("Could not save config file.");
      std::cerr << "Could not save config file." << std::endl;
      return ;
    }
  for (const auto& it: Config::values)

M louloulibs/config/config.hpp => louloulibs/config/config.hpp +0 -1
@@ 15,7 15,6 @@

#pragma once


#include <functional>
#include <fstream>
#include <memory>

M tests/config.cpp => tests/config.cpp +1 -1
@@ 8,7 8,7 @@
TEST_CASE("Config basic")
{
  // Disable all output for this test
  IoTester<std::ostream> out(std::cout);
  IoTester<std::ostream> out(std::cerr);
  // Write a value in the config file
  Config::read_conf("test.cfg");
  Config::set("coucou", "bonjour", true);