Currently the "ConquestCritical" class is added to the basic template_structure.xml file and subsequently removed in the majority of cases. This is quite ugly. A much neater way is to add it only to the cases that actually have it (Civic, Military, Wonder, and several special structures), but not to the basic parent template.
Effectively nothing ought to be changed.
15 removed cases, 10 added ones (if counting holds, the 15 goes down to 13 given the structure_defense_* comment)
Not convinced either way about this patch
With the current design the structure_defense_* could be moved into the parent
Currently template_structure.xml adds ConquestCritical, but:
remove it. Instead of adding it everywhere and subsequently removing it often, I think it's neater to simply add it only to the cases that actually use it:
- several structures/ files.