refactoring

Author: SandrineP

src/main.cpp

@@ -2,22 +2,24 @@
 #include <git2.h>  // For version number only
 #include <iostream>
 
-#include "git_exception.hpp"
+#include "src/utils/git_exception.hpp"
 #include "version.hpp"
 #include "subcommand/init_subcommand.hpp"
+// #include "subcommand/status_subcommand.hpp"
 
 int main(int argc, char** argv)
 {
     int exitCode = 0;
     try
     {
+        const libgit2_object lg2_obj;
         CLI::App app{"Git using C++ wrapper of libgit2"};
 
         // Top-level command options.
         auto version = app.add_flag("-v,--version", "Show version");
 
         // Sub commands
-        InitSubcommand init(app);
+        InitSubcommand init(lg2_obj, app);
 
         app.parse(argc, argv);
 

src/meson.build

@@ -1,7 +1,7 @@
 subdir('subcommand')
+subdir('utils')
 subdir('wrapper')
 
 src_files = files([
-    'git_exception.cpp',
     'main.cpp'
-]) + subcommand_files + wrapper_files
+]) + subcommand_files + utils_files + wrapper_files

src/subcommand/base_subcommand.hpp

@@ -1,8 +1,7 @@
 #include <filesystem>
 #include "init_subcommand.hpp"
-#include "../wrapper/repository_wrapper.hpp"
 
-InitSubcommand::InitSubcommand(CLI::App& app)
+InitSubcommand::InitSubcommand(const libgit2_object&, CLI::App& app)
 {
     auto *sub = app.add_subcommand("init", "Explanation of init here");
 
@@ -18,6 +17,5 @@ InitSubcommand::InitSubcommand(CLI::App& app)
 
 void InitSubcommand::run()
 {
-    RepositoryWrapper repo;
-    repo.init(directory, bare);
+    RepositoryWrapper::init(directory, bare);
 }

src/subcommand/init_subcommand.cpp

@@ -1,12 +1,15 @@
 #pragma once
 
+#include <CLI/CLI.hpp>
 #include <string>
-#include "base_subcommand.hpp"
 
-class InitSubcommand : public BaseSubcommand
+#include "src/wrapper/repository_wrapper.hpp"
+
+class InitSubcommand
 {
 public:
-    InitSubcommand(CLI::App& app);
+
+    explicit InitSubcommand(const libgit2_object&, CLI::App& app);
     void run();
 
 private:

src/subcommand/init_subcommand.hpp

@@ -0,0 +1,13 @@
+#include <git2.h>
+
+#include "common.hpp"
+
+libgit2_object::libgit2_object()
+{
+    git_libgit2_init();
+}
+
+libgit2_object::~libgit2_object()
+{
+    git_libgit2_shutdown();
+}

src/utils/common.cpp

@@ -0,0 +1,20 @@
+class noncopiable_nonmovable
+{
+public:
+    noncopiable_nonmovable(const noncopiable_nonmovable&) = delete;
+    noncopiable_nonmovable& operator=(const noncopiable_nonmovable&) = delete;
+    noncopiable_nonmovable(noncopiable_nonmovable&) = delete;
+    noncopiable_nonmovable& operator=(noncopiable_nonmovable&&) = delete;
+
+protected:
+    noncopiable_nonmovable() = default;
+    ~noncopiable_nonmovable() = default;
+};
+
+class libgit2_object : private noncopiable_nonmovable
+{
+public:
+
+    libgit2_object();
+    ~libgit2_object();
+};

src/utils/common.hpp

@@ -0,0 +1,4 @@
+utils_files = files([
+    'common.cpp',
+    'git_exception.cpp',
+])