fix: when compiling code for the debugger, set is_result_unused to true

Author: SuperFola

CHANGELOG.md

@@ -3,6 +3,10 @@
 ## [Unreleased changes] - 2026-MM-DD
 ### Breaking changes
 
+### Deprecations
+- `list:permutations` is deprecated in favor of `list:combinations`
+- `list:permutationsWithReplacement` is deprecated in favor of `list:combinationsWithReplacement`
+
 ### Added
 - new debugger commands: `stack <n>` and `locals <n>` to print the values on the stack and in the current locals scope
 - custom format specifiers for lists:

src/arkreactor/Compiler/Lowerer/ASTLowerer.cpp

@@ -38,8 +38,8 @@ namespace Ark::internal
         compileExpression(
             ast,
             /* current_page */ global,
-            /* is_result_unused */ false,
-            /* is_terminal */ false);
+            /* is_result_unused= */ true,
+            /* is_terminal= */ false);
         m_logger.traceEnd();
     }
 

tests/unittests/Suites/BytecodeReaderSuite.cpp

@@ -121,7 +121,7 @@ ut::suite<"BytecodeReader"> bcr_suite = [] {
         should("list all code page") = [inst_locations_block, pages, start_code] {
             expect(that % start_code == inst_locations_block.end);
             expect(that % pages.size() == 2ull);
-            expect(that % pages[0].size() == 9 * 4ull);
+            expect(that % pages[0].size() == 10 * 4ull);
             expect(that % pages[1].size() == 20 * 4ull);
         };
     };

tests/unittests/Suites/DiagnosticsSuite.cpp

@@ -38,9 +38,11 @@ ut::suite<"Diagnostics"> diagnostics_suite = [] {
         "DiagnosticsSuite/runtime",
         [](TestData&& data) {
             Ark::State state({ lib_path });
+            // custom output stream for warnings to hide them, we don't want to test them here
+            std::stringstream stream;
 
             should("compile without error runtime/" + data.stem) = [&] {
-                expect(mut(state).doFile(data.path, features));
+                expect(mut(state).doFile(data.path, features, &stream));
             };
 
             should("generate an error at runtime in " + data.stem) = [&] {
@@ -64,9 +66,11 @@ ut::suite<"Diagnostics"> diagnostics_suite = [] {
         "DiagnosticsSuite/typeChecking",
         [](TestData&& data) {
             Ark::State state({ lib_path });
+            // custom output stream for warnings to hide them, we don't want to test them here
+            std::stringstream stream;
 
             should("compile without error typeChecking/" + data.stem) = [&] {
-                expect(mut(state).doFile(data.path, features));
+                expect(mut(state).doFile(data.path, features, &stream));
             };
 
             should("generate an error at runtime (typeChecking) in " + data.stem) = [&] {

tests/unittests/resources/CompilerSuite/ir/ackermann.expected

@@ -12,6 +12,7 @@ page_0
 	BUILTIN 9
 	CALL 2
 .L5:
+	POP 0
 	HALT 0
 
 page_1

tests/unittests/resources/CompilerSuite/ir/args_attr.expected

@@ -16,6 +16,7 @@ page_0
 	BUILTIN 9
 	CALL 1
 .L0:
+	POP 0
 	HALT 0
 
 page_1

tests/unittests/resources/CompilerSuite/ir/breakpoints.expected

@@ -50,6 +50,7 @@ page_0
 	LOAD_FAST_BY_INDEX 0
 	CALL 0
 .L9:
+	POP 0
 	HALT 0
 
 page_1

tests/unittests/resources/CompilerSuite/ir/closures.expected

@@ -91,6 +91,7 @@ page_0
 	BUILTIN 9
 	CALL 2
 .L12:
+	POP 0
 	HALT 0
 
 page_1

tests/unittests/resources/CompilerSuite/ir/factorial.expected

@@ -11,6 +11,7 @@ page_0
 	BUILTIN 9
 	CALL 2
 .L2:
+	POP 0
 	HALT 0
 
 page_1

tests/unittests/resources/CompilerSuite/ir/operators.expected

@@ -69,4 +69,5 @@ page_0
 	BUILTIN 9
 	CALL 22
 .L0:
-	HALT 0
+	POP 0
+	HALT 0