Currently ruledit validity.[ch] provides set of functions named "is_..._needed()". It's correct from ruledit PoV, but not all of freeciv. Especially if we move the module to common/ ( #41652 considers this ), we should distinguish the naming for the semantic difference to some functions there that work from player's PoV. Even if some other ruleset object refers to an object, it might not be providing anything useful, or vice versa (it can provide things by a xxxFlag requirement even if no requirement refers directly to it)
Currently ruledit validity.[ch] provides set of functions named "is_..._needed()". It's correct from ruledit PoV, but not all of freeciv. Especially if we move the module to common/ ( #41652 considers this ), we should distinguish the naming for the semantic difference to some functions there that work from player's PoV. Even if some other ruleset object refers to an object, it might not be providing anything useful, or vice versa (it can provide things by a xxxFlag requirement even if no requirement refers directly to it)