Merge pull request #11 from st235/feature/pedantic_warnings

Enable pedantic warnings

CMakeLists.txt

@@ -16,6 +16,12 @@ add_library(jsonc
     src/json.cpp
 )
 
+if(MSVC)
+  target_compile_options(jsonc PRIVATE /W4 /WX)
+else()
+  target_compile_options(jsonc PRIVATE -Wall -Wextra -Wpedantic -Werror)
+endif()
+
 set_property(TARGET jsonc PROPERTY COMPILE_WARNING_AS_ERROR ON)
 
 target_include_directories(jsonc PUBLIC ${CMAKE_CURRENT_LIST_DIR}/include)

include/json.h

@@ -377,7 +377,7 @@ class Json {
         keys.insert(key);
       }
 
-      return std::move(keys);
+      return keys;
     }
 
     bool add(const Json& that) {

include/json_beautifier.h

@@ -45,7 +45,7 @@ class JsonBeautifier: public JsonVisitor {
     }
 
     inline std::string beautifiedJson() const {
-        return std::move(_stream.str());
+        return _stream.str();
     }
 
     uint16_t _depth;

include/json_minifier.h

@@ -33,7 +33,7 @@ class JsonMinifier: public JsonVisitor {
     }
 
     inline std::string minifiedJson() const {
-        return std::move(_stream.str());
+        return _stream.str();
     }
 
     std::stringstream _stream;

src/json.cpp

@@ -12,21 +12,21 @@ const Json& Json::null() {
 }
 
 Json Json::array() {
-    return std::move(Json(array_t()));
+    return Json(array_t());
 }
 
 Json Json::object() {
-    return std::move(Json(object_t()));
+    return Json(object_t());
 }
 
 std::optional<Json> Json::fromJson(const std::string& json) {
     __internal::JsonParser parser;
-    return std::move(parser.parse(json));
+    return parser.parse(json);
 }
 
 std::string Json::toJson(const Json& json) {
     JsonMinifier minifier;
-    return std::move(minifier.minify(json));
+    return minifier.minify(json);
 }
 
 } // namespace json