Page MenuHomePhorge

TransactionEditor: micro-optimize the "creation finder"
ClosedPublic

Authored by valerio.bozzolan on May 9 2024, 08:32.
Tags
None
Referenced Files
F3576225: D25627.1745034883.diff
Fri, Apr 18, 03:54
F3554833: D25627.1744991740.diff
Thu, Apr 17, 15:55
F3389907: D25627.1744461256.diff
Fri, Apr 11, 12:34
F3372746: D25627.1744251834.diff
Wed, Apr 9, 02:23
F3369764: D25627.1744216319.diff
Tue, Apr 8, 16:31
F3368296: D25627.1744207467.diff
Tue, Apr 8, 14:04
F3367435: D25627.1744199949.diff
Tue, Apr 8, 11:59
F3359976: D25627.1743967457.diff
Sat, Apr 5, 19:24

Details

Summary

The foreach starting at old line 1305-1311 was designed to look for a specific transaction type.

When we have found that, let's stop that loop, so, we save some CPU cycles.

Test Plan

Note that getTransactionType() has not side-effects.

Test some applications - like Pholio - that use the TransactionEditor:

  • Create a new Pholio and Save
  • Save it again without changes.
  • Save it again with changes.

Everything still working normally.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable