GetNodesInRegion possible bug

General
1

Hello. Currently updating to v 4.0.6 from 3.8.6. I have a call to GetNodesInRegion where the bounds are such that the rect from the GetRectFromBounds call has a negative height. This can result in a ListPool Claim call with a negative capacity and thus an error.

Original:
var inArea = ListPool.Claim(rect.Width*rect.Height);

My fix:
var inArea = ListPool.Claim(Mathf.Abs(rect.Width*rect.Height));

Does this seem about right or should this never be sent a bounds which can result in a negative width or height in rect form?

2

Hi

Thank you for reporting this bug.
The proper way for that method to behave would be to simply return an empty list in those cases.

I have fixed this in my dev version now, I will likely release an update later today with the fix.