fix more clang-tidy warnings

Author: pantor

include/inja/renderer.hpp

@@ -350,7 +350,7 @@ class Renderer : public NodeVisitor {
     } break;
     case Op::Default: {
       const auto test_arg = get_arguments<1, 0, false>(node)[0];
-      data_eval_stack.push(test_arg ? test_arg : get_arguments<1, 1>(node)[0]);
+      data_eval_stack.push((test_arg != nullptr) ? test_arg : get_arguments<1, 1>(node)[0]);
     } break;
     case Op::DivisibleBy: {
       const auto args = get_arguments<2>(node);
@@ -425,7 +425,7 @@ class Renderer : public NodeVisitor {
       const auto precision = args[1]->get<const json::number_integer_t>();
       const double result = std::round(args[0]->get<const json::number_float_t>() * std::pow(10.0, precision)) / std::pow(10.0, precision);
       if (precision == 0) {
-        make_result(int(result));
+        make_result(static_cast<int>(result));
       } else {
         make_result(result);
       }
@@ -557,7 +557,7 @@ class Renderer : public NodeVisitor {
     additional_data[static_cast<std::string>(node.value)].clear();
     if (!(*current_loop_data)["parent"].empty()) {
       const auto tmp = (*current_loop_data)["parent"];
-      *current_loop_data = std::move(tmp);
+      *current_loop_data = tmp;
     } else {
       current_loop_data = &additional_data["loop"];
     }

include/inja/statistics.hpp

@@ -63,9 +63,9 @@ class StatisticsVisitor : public NodeVisitor {
   void visit(const SetStatementNode&) override {}
 
 public:
-  size_t variable_counter;
+  size_t variable_counter {0};
 
-  explicit StatisticsVisitor(): variable_counter(0) {}
+  explicit StatisticsVisitor() {}
 };
 
 } // namespace inja

single_include/inja/inja.hpp

@@ -807,9 +807,9 @@ class StatisticsVisitor : public NodeVisitor {
   void visit(const SetStatementNode&) override {}
 
 public:
-  size_t variable_counter;
+  size_t variable_counter {0};
 
-  explicit StatisticsVisitor(): variable_counter(0) {}
+  explicit StatisticsVisitor() {}
 };
 
 } // namespace inja
@@ -2503,7 +2503,7 @@ class Renderer : public NodeVisitor {
     } break;
     case Op::Default: {
       const auto test_arg = get_arguments<1, 0, false>(node)[0];
-      data_eval_stack.push(test_arg ? test_arg : get_arguments<1, 1>(node)[0]);
+      data_eval_stack.push((test_arg != nullptr) ? test_arg : get_arguments<1, 1>(node)[0]);
     } break;
     case Op::DivisibleBy: {
       const auto args = get_arguments<2>(node);
@@ -2578,7 +2578,7 @@ class Renderer : public NodeVisitor {
       const auto precision = args[1]->get<const json::number_integer_t>();
       const double result = std::round(args[0]->get<const json::number_float_t>() * std::pow(10.0, precision)) / std::pow(10.0, precision);
       if (precision == 0) {
-        make_result(int(result));
+        make_result(static_cast<int>(result));
       } else {
         make_result(result);
       }
@@ -2710,7 +2710,7 @@ class Renderer : public NodeVisitor {
     additional_data[static_cast<std::string>(node.value)].clear();
     if (!(*current_loop_data)["parent"].empty()) {
       const auto tmp = (*current_loop_data)["parent"];
-      *current_loop_data = std::move(tmp);
+      *current_loop_data = tmp;
     } else {
       current_loop_data = &additional_data["loop"];
     }