fix(ExecutionTrace): fixed unintentional override of currentOp in get… (

#77) Ok lets merge it for now
nebulastream · Jan 15, 2025 · ea13704 · ea13704
1 parent 542d8af
commit ea13704
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/nautilus/src/nautilus/tracing/ExecutionTrace.cpp b/nautilus/src/nautilus/tracing/ExecutionTrace.cpp
@@ -133,13 +133,11 @@ void ExecutionTrace::nextOperation() {
 }
 
 TraceOperation& ExecutionTrace::getCurrentOperation() {
-	auto& currentOp = getCurrentBlock().operations[currentOperationIndex];
-	while (currentOp.op == JMP) {
-		auto& nextBlock = std::get<BlockRef>(currentOp.input[0]);
+	while (getCurrentBlock().operations[currentOperationIndex].op == JMP) {
+		auto& nextBlock = std::get<BlockRef>(getCurrentBlock().operations[currentOperationIndex].input[0]);
 		setCurrentBlock(nextBlock.block);
-		currentOp = getCurrentBlock().operations[currentOperationIndex];
 	}
-	return currentOp;
+	return getCurrentBlock().operations[currentOperationIndex];
 }
 
 uint16_t ExecutionTrace::createBlock() {