evaluator: Fix more warnings

WerWolv · Feb 1, 2025 · 3095ad4 · 3095ad4
1 parent 506b867
commit 3095ad4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/include/pl/core/evaluator.hpp b/lib/include/pl/core/evaluator.hpp
@@ -78,7 +78,7 @@ namespace pl::core {
         struct Scope {
             Scope(const std::shared_ptr<pl::ptrn::Pattern>& parentPattern,
                 std::vector<std::shared_ptr<pl::ptrn::Pattern>>* scopePatterns,
-                unsigned long heapSize)
+                size_t heapSize)
                 : parent(parentPattern),
                 scope(scopePatterns),
                 heapStartSize(heapSize) {}

diff --git a/lib/source/pl/core/ast/ast_node_mathematical_expression.cpp b/lib/source/pl/core/ast/ast_node_mathematical_expression.cpp
@@ -194,11 +194,17 @@ namespace pl::core::ast {
                    case Token::Operator::Slash:
                        if (right == 0)
                            err::E0002.throwError("Division by zero.", { }, this->getLocation());
-                       return new ASTNodeLiteral(R(R(left) / R(right)));
+                       if constexpr (std::same_as<R, bool>)
+						   err::E0001.throwError("Cannot divide boolean values.", { }, this->getLocation());
+                       else
+                           return new ASTNodeLiteral(R(R(left) / R(right)));
                    case Token::Operator::Percent:
                        if (right == 0)
                            err::E0002.throwError("Division by zero.", { }, this->getLocation());
-                       return new ASTNodeLiteral(R(modulus(left, right)));
+                       if constexpr (std::same_as<R, bool>)
+                           err::E0001.throwError("Cannot divide boolean values.", { }, this->getLocation());
+                       else
+                           return new ASTNodeLiteral(R(modulus(left, right)));
                    case Token::Operator::LeftShift:
                        return new ASTNodeLiteral(R(shiftLeft(left, right)));
                    case Token::Operator::RightShift: