Merge pull request #32 from pmeerw/cleanup

Some cleanup
build-cpp · Dec 10, 2021 · a28e63f · a28e63f
2 parents 1c23469 + 31cc1c9
commit a28e63f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/src/build.cpp b/src/build.cpp
@@ -5,9 +5,9 @@
 
 #include "fs.hpp"
 #include <sstream>
-#include <stddef.h>
+#include <cstddef>
 #include <stdexcept>
-#include <stdlib.h>
+#include <cstdlib>
 #include <system_error>
 
 namespace cmkr {
@@ -94,4 +94,4 @@ int cmkr_build_install(void) {
     } catch (...) {
         return cmkr::error::Status(cmkr::error::Status::Code::InstallError);
     }
-}
+}
diff --git a/src/error.cpp b/src/error.cpp
@@ -1,6 +1,6 @@
 #include "error.hpp"
 
-#include <assert.h>
+#include <cassert>
 
 namespace cmkr {
 namespace error {
@@ -14,11 +14,12 @@ Status::Code Status::code() const noexcept { return ec_; }
 } // namespace error
 } // namespace cmkr
 
-const char *err_string[] = {
-    "Success", "Runtime error", "Initialization error", "CMake generation error", "Build error",
+// strings for cmkr::error::Status::Code
+static const char *err_string[] = {
+    "Success", "Runtime error", "Initialization error", "CMake generation error", "Build error", "Clean error", "Install error",
 };
 
 const char *cmkr_error_status(int i) {
-    assert(i >= 0 && i < 5);
+    assert(i >= 0 && i < (sizeof(err_string) / sizeof(*(err_string))));
     return err_string[i];
-}
+}