UKismetStringLibrary::CullArray is poorly implemented

FString::CullArray is a static function, but the BP version is still taking a string instance to operate on (which it won't mutate as the function is static), and takes the array as an in/out argument (which doesn't really work with BPs).

The BP exposed function should probably be changed to take an array, internally make a copy, call FString::CullArray on the copy, and then return the result.

Something like this:

TArray<FString> UKismetStringLibrary::CullArray(const TArray<FString>& InArray)
{
	TArray<FString> CulledArray = InArray;
	FString::CullArray(&CulledArray);
	return CulledArray;
}

1. Create a "Cull Array" node in a BP

How does 'cull array' work?