Follow-up Comment #2, patch #1141 (project wesnoth):
Yeah, there is a lot of wasted space int quadtree::tree. It was just easier
to do it this way for testing. That will definitely be changed.
You are right that "+ 1" would make much more sense for the halting
condition. And why 48? No reason, that condition could use some more testing
and tweaking.
The final rectangle calculation is currently a bit sketchy. The extra -1 is
because I believe that the rectangles we send to SDL are inclusive. Sending
rectangles with x + tw == w or y + th == h is for some reason very slow,
which I believe again is due to the rectangles being inclusive and then
having a part that is actually off the screen.
And finally, yeah the skipping the column we just erased was something that
did not make it from my head to the screen as I wrote that code.
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?1141>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Wesnoth-bugs mailing list
Wesnoth-bugs@gna.org
https://mail.gna.org/listinfo/wesnoth-bugs
