Incorrect number of parameters in empty Shipping version of the DrawDebugString() function signature

The DrawDebugString() function has to versions. One is the actual function to display a string of debug text in non-Shipping builds, and the other is an empty function that is used in Shipping builds. However, the signature for each of these versions of the function is different. The normal function takes 8 parameters, but the empty Shipping version of the function only takes 7 parameters. This will result in a Shipping build failing if this function is called with 8 parameters (and it is not guarded by something like #if !UE_BUILD_SHIPPING).

REGRESSION:
No. The same behavior occurs in 4.20.3.

1. Create a new Basic Code project.
2. Add a new Actor class to the project.
3. Include the DrawDebugHelpers.h file in the new Actor's .cpp file.
4. In one of the included functions of the new Actor class (eg. BeginPlay()), add a call to DrawDebugString() using all 8 parameters.
   • eg: DrawDebugString(GetWorld(), FVector(0.f, 0.f, 0.f), TEXT("This is my test text."), nullptr, FColor::MakeRandomColor(), 1.f, false, 1.f);
5. Build the project for DevelopmentEditor.
6. Once the first build completes, build the project for Shipping.

RESULT:
The Shipping build fails with an error stating that DrawDebugString() does not take 8 parameters.

EXPECTED:
The Shipping build completes successfully.