Skip to content

Conversation

@arturjpv
Copy link
Member

Developers are having problems if they don't place the ifdef right after the check(System::IsServer()), as the server scope is compared starting after the check and we fail to detect that the server scope is already protected.

	UFUNCTION(BlueprintOverride)
	void Tick(float DeltaSeconds)
	{
		FTraceScope TraceScope(n"UGaitGroundContactHelperComponent::Tick");
		check(System::IsServer());

#ifdef WITH_SERVER
		GroundContactHitResults.Empty();
#endif
	}

Build Discovery.
Build Pioneer.

@arturjpv arturjpv requested a review from tvandijck as a code owner February 10, 2025 15:30
@arturjpv arturjpv merged commit d6573b3 into main Feb 10, 2025
1 check passed
@arturjpv arturjpv deleted the feature/allow-empty-lines branch February 11, 2025 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants