beyond-all-reason / spring

A powerful free cross-platform RTS game engine
https://beyond-all-reason.github.io/spring/
Other
210 stars 100 forks source link

Remove `tdfID` weapon def #1467

Open sprunk opened 5 months ago

sprunk commented 5 months ago

Replaced with regular ID in COB, removed elsewhere.

marcushutchings commented 5 months ago

So how would games track weaponDef ids? If they get hit by weapon id 20, how do they know what weapon that is?

sprunk commented 5 months ago

They still get the weapon def id. This patch removes tdf id.

sprunk commented 5 months ago

No known game sets the tag: tdfid path:*.lua tdfid path:*.tdf

marcushutchings commented 5 months ago

Does anything else need to be removed from the engine if tdf files are not going to be supported?

sprunk commented 5 months ago

This doesn't affect tdf file support, only the tdfID weapon def, which only has tdf in the name and is otherwise unrelated to tdf files.