Iterating twice?
Bug #1166166 reported by
Jiri Srba
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
VerifyDTAPN |
In Progress
|
Undecided
|
Unassigned |
Bug Description
Referring to line numbers in the merge proposal
https:/
Line 3489-3502: This loop should really be an else statement to the condition in line 3480-3486. Currently, we iterate over all tokens to test if any are older than c_max even though we just removed all tokens older than c_max --> twice the iterations for dead places (I know this is small, but because it is in cut it will be run a lot).
To post a comment you must log in.
Is there any reason for that at all, can the code between 3480 - 3486 not just be omitted?