filter: use the O(log(n)) instead of O(n) method #78569
Merged
+4
−7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
None
Purpose of change
I used O(n) and complicated the way of getting
itype
fromitype_id
in:L:
,V:
,M:
(longest_side, volume, mass); affects AIM, inventory… #78519Describe the solution
Use the correct and faster method.
Describe alternatives you've considered
Testing
L:122 cm
in AIM and crafting GUI.Additional context
L:
,V:
,M:
(longest_side, volume, mass); affects AIM, inventory… #78519 but I didn't know I just missed the correct method.