There is a possible code mistake in UPrimitiveComponent::GetCollisionShape.

A user pointed out a possible mistake in the Engine source code. In the GetCollisionShape function in the UPrimitiveComponent class, an FVector variable Extent is created and set, and then potentially modified to make sure it is not smaller than zero size. However, this variable is never actually used.

Looking over the code, I suspect that it is intended for the return line to be return FCollisionShape::MakeBox(Extent);

Edit Tim L
========
Another user voiced the concern that the if statement in the function should be checking Extent instead of the float value that is passed into the function. This should only be a concern if we do not check Bounds.BoxExtent anywhere else to make sure it never contains a negative value. From what I can tell, this value should default to ZeroVector, but I could not find anything that would make sure none of the values were ever set to a negative value.

1. Open the PrimitiveComponent.cpp file in the source code.
2. Locate the GetCollisionShape function.
   • In CL-2736016 it is located at line 1394.

RESULT:
The Extent variable is created and set, but never used.

EXPECTED:
The Extent variable is used.