From 4932138fba7ff8e1ac12d607f8b8bd9abe3113f0 Mon Sep 17 00:00:00 2001
From: ado <adnan.abd0124@gmail.com>
Date: Sun, 17 Jan 2021 15:22:50 +0100
Subject: [PATCH] fix parser destructor

---
 include/ss/parser.hpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/include/ss/parser.hpp b/include/ss/parser.hpp
index 60c4758..6a4e807 100644
--- a/include/ss/parser.hpp
+++ b/include/ss/parser.hpp
@@ -3,9 +3,9 @@
 #include "converter.hpp"
 #include "extract.hpp"
 #include "restrictions.hpp"
+#include <cstdlib>
 #include <cstring>
 #include <optional>
-#include <cstdlib>
 #include <string>
 #include <vector>
 
@@ -29,7 +29,9 @@ public:
     }
 
     ~parser() {
-        fclose(file_);
+        if (file_) {
+            fclose(file_);
+        }
     }
 
     bool valid() const {
@@ -135,8 +137,8 @@ public:
             auto merged_values =
                 std::tuple_cat(std::move(values_),
                                std::tuple<T>{parser_.valid()
-                                              ? std::forward<T>(new_value)
-                                              : std::nullopt});
+                                                 ? std::forward<T>(new_value)
+                                                 : std::nullopt});
             return {std::move(merged_values), parser_};
         }