Remove useless change

Merge snd block, update message, fix compilation, (Config 2 change is just for the CI)

In D2810#132054, @bb wrote:

Observers can already see the whole map, slightly bad that observers don't have the retroMe cheat anymore, but well, meh.

D1442 for the record

It's important in the sense that it removes decoration as such makes buildings more historically accurate, but also more bland and less interesting to look at. So when I need to edit buildings I have to take it into account.

That also removes the two characters from the walls

Will that break Atlas animation hotloading?

Maybe we should allow specific resources to go negative, eg if one wants to use coin.

Did you svn add tree/ ?

http://irclogs.wildfiregames.com/2020-09/2020-09-11-QuakeNet-%230ad-dev.log

The Brit one did ?

@wraitii , @bb, @Angen ?

(That's a tower)

@wraitii @Freagarach can you have another look at this ?

@Angen not sure what you mean? Can't you call it when you display the message ?

Can we support playing a sound ?

Now D2998

Wonder if we shouldn't use CStr but likely out of the scope of that patch.

That means you can't build anymore units if you only have a barracks left ?

Could be nice to keep the flags on just in case somebody has sillouetes off.

Is that the only function that can be replaced by a member ? (i didn't check)

Remove the class keyword?

Are there any of the function calls that can be replaced by member access?

I'd like to have Itms input on this. Not a fan of the capped FPS too.

You might want to delete ICmpSettlement.cpp/h while at it.

@aeonios feel free to commandeer this patch as it's low priority on @vladislavbelov's todo list.

@Gallaecio if you have some time

Not currently.

The good thing is playsound just does nothing if there is no soundgroup in the template so you can add it with little extra code.

Can we play a sound when the unit refuses to move ?

@Freagarach any thought about the tower renaming ?

(Courtesy of Alexandermb)

That unit exists and has been updated recently in mods. If you want it I can add it to vanilla

Yes, just try the patcg my friend :)

Looks like Z Fighting

The trees are ignored for performance.

I just tested the patch, it works fine on my Intel UHD and on my GTX 1070. I noticed it totally ignores trees, but I guess that's intended.

Well most of the functionnality was removed on vlad's feedback, and I'm not happy about it wraitii also said it was not worth it in that case, so i'd rather not work on something that doesn't make me want to commit it. And so no nothing is planned.

Neat. I guess @vladislavbelov might want to look at it.

For posterity (Discussion in a PM)

Not worth it anymore.

remove flush code from the main loop
fix thread disposal
fix incorrect function call getguid
fix segfault when starting the game

I think @vladislavbelov might have better input.

Well it's not a priority but it a nice feature for testing audio :)

Fixes my issue on Intel with GLSL on
Before:

You mean the top panel? I didn't touch that.

No, I mean the player color band below the unit name.

In D2936#128282, @OptimusShepard wrote:

In D2936#128281, @Stan wrote:

Dying animation is the corpse ^^

Ok. Is it possible, to splitt the corpse animation into dying and decay animations, and than remove only the decay animation? I guess, only the long decay animation is performance critical?

In D2936#128279, @OptimusShepard wrote:

A dying animation is not possible here? Or at least a simple visual plop effect?

I just tested this, why did you reduce the band with the civ emblem ? The minimap buttons no longer fit the circle (as in they have a bigger gap), the bottom bells are now bigger than previously ?

@vladislavbelov do your thang.

Make it black and white. Apparently storing into a dequeue has a too big performance impact.