Comment on Cost vs costFunc

algorand · Mar 26, 2022 · 8256400 · 8256400
1 parent 2aef60e
commit 8256400
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/data/transactions/logic/eval.go b/data/transactions/logic/eval.go
@@ -907,6 +907,10 @@ func (cx *EvalContext) step() error {
 
 	opcost := deets.Cost
 	if opcost == 0 {
+		// We use a constant (deets.Cost) to avoid the cost of an indirect
+		// function call in most cases.  Consider getting the cost details into
+		// OpSpec so that a single (inlinable?) function can compute the cost,
+		// rather than a per opcode function pointer.
 		opcost = deets.costFunc(cx.program, cx.pc)
 	}
 	if opcost == 0 {