Simplify code and remove duplicate loop to find exact matches
ClosedPublic

Authored by mwolff on Apr 4 2018, 11:33 PM.

Details

Summary

Sadly, the code is not any faster (nor slower) - according to
the benchmark the performance stays pretty much the same. Still,
it's worth it to merge the loop for readability I think. It clearly
shows that we want to go through the segments once to find a
certain match quality.

Diff Detail

Repository
R32 KDevelop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mwolff created this revision.Apr 4 2018, 11:33 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptApr 4 2018, 11:33 PM
mwolff requested review of this revision.Apr 4 2018, 11:33 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 9 2018, 6:31 PM
This revision was automatically updated to reflect the committed changes.