GetHexCentersWithinSquare bugged

Hello.

Just got the framework and started playing with it. I used GetHexCentersWithinSquare to get all the hexes in a rectangle in order to create arectangular map.
To my surprise, the map created is NOT rectangular.
The culprit is the Rect class, Contains method. It behaves badly! If you're on the left or bottom border, the check uses <= and so you're in. If you're on the right or top border of the rectangle, then the check uses < and you're out.
So generating a rectangle patch looks like this (to test, use a 6x4 rect or 30x20 if you like to see it big):


-xxxx-

xxxxx-

xxxxx-



I had to change the code this way:



// if (r.Contains(HexCoordinates.HexToWorld(pos))) -- old code

var worldPos = HexCoordinates.HexToWorld(pos);

if( r.x <= worldPos.x && r.x + r.width >= worldPos.x && r.y <= worldPos.y && r.y + r.height >= worldPos.y)



in order for it to behave better. This gives at least a symmetrical look:



-xxxxx
xxxxxx
xxxxx-

Well, I'd expect to actually get a rectangle, like that:
xxxxx
xxxxx
xxxxx
I can find out why there's a discrepancy along the 0 axis,but the method should be symmetric and it's currently not because Rect.Contains has been badly coded. You might want to check it and replace the calls to Rect by calls to a Rect that manages both ends of the rectangle in the same way.