Page MenuHomeWildfire Games

Update range queries to account for entity size
Needs ReviewPublic

Authored by wraitii on Fri, May 22, 1:03 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Range query checks currently do not take entity size into account. This can lead to returning too few entities and other unexpected behaviour.

This change simply adds size over-optimistically.

It likely leads to a slight performance decrease as this slightly increases all our ranges.


Discovered following Angen's comment here.

Test Plan

Agree that the change is desirable.

Event Timeline

wraitii created this revision.Fri, May 22, 1:03 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2199/display/redirect

Commenting to myself that I need to check what this does for gates.

Freagarach edited the summary of this revision. (Show Details)Thu, Jun 4, 8:28 PM