mentos987
mentos987
Vultraz has requested that this wait until after SDL3 merge, otherwise it is ready to merge.
@doofus-01 Is this kind of solution satisfactory to you? The bug https://github.com/wesnoth/wesnoth/issues/6203 will require that we apply these fixes to a fair few units. I estimate about 30 units.
@doofus-01 > I'm not sure about the high layer for huge units, but status quo isn't great either. Only the middle hex gets moved up above foreground terrain here. (Look...
@doofus-01 Wyverns are also put on foreground layer. And I think that it will be good to open up the option of having huge "boss" units.
I think this one is done.
> The `advance_to_current_frame()` results seem odd. It looks like it got worse? Averages are ~2 before and ~7 after. No, it got way better. The reason why the average call...
> it'd be good to include the total runtime to do the same work as well then. It is the same pretty much, the only other extra runtime added is...
I can separate out all the comments and the halo optimising into separate commits if you want. But following three points are very entangled (hard top split up): 1. Renaming...
My next steps will be: 1. Test for more bugs. 2. Make new tests to share here that covers a more complete picture of the change. Likely by going upstream...
Here follows a more complete test of a heavy load scenario (see image). The test is done by letting screen stay like the image over a minute and printing data...