fix(vm): off-by-one in code block stringification.

The call to retrieve operands modifies pc, setting it to the index of the *next* instruction. So, we save its initial value and use that for printing. Acked-by: Taylor Sutton <[email protected]>
boa-dev · Apr 2, 2022 · 82e9652 · 82e9652
1 parent a5337a9
commit 82e9652
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/boa_engine/src/vm/code_block.rs b/boa_engine/src/vm/code_block.rs
@@ -351,9 +351,10 @@ impl ToInternedString for CodeBlock {
         while pc < self.code.len() {
             let opcode: Opcode = self.code[pc].try_into().expect("invalid opcode");
             let opcode = opcode.as_str();
+            let previous_pc = pc;
             let operands = self.instruction_operands(&mut pc, interner);
             f.push_str(&format!(
-                "{pc:06}    {count:04}    {opcode:<27}{operands}\n",
+                "{previous_pc:06}    {count:04}    {opcode:<27}{operands}\n",
             ));
             count += 1;
         }