Description

When doing overlap tests in PrimitiveComponent, UWorld->ComponentOverlapMulti is used.

The callstack for that ends up looking like this (callstack is inverted, highest level on top):

UWorld::ComponentOverlapMulti(...)
UWorld::ComponentOverlapMultiByChannel(...)
UPrimitiveComponent::ComponentOverlapMulti(...)
UPrimitiveComponent::ComponentOverlapMultiImple(...)

The first two calls (in UWorld) don't add anything, other than default ComponentQueryParams (which is FComponentQueryParams::DefaultComponentQueryParams). Further, they both perform null checks on the input PrimitiveComponent (which are unnecessary given it is this).

This default could easily just be added to the primitive component, and the call made directly.

Steps to Reproduce

Test ComponentOverlapMulti still works.
This change doesn't change functionality or fix a bug, instead just cleans up code.

Community References

Have Comments or More Details?

There's no existing public thread on this issue, so head over to Questions & Answers just mention UE-36472 in the post.

0
Login to Vote

Fixed
Fix Commit3172152
Main Commit3182044
CreatedSep 27, 2016
ResolvedOct 24, 2016
UpdatedJun 23, 2018