Hey, everyone! Per recently committed a template overhaul to trunk. It was very nice, and improved template loading significantly (though I think it may the cause of the new borg template bugs), but there's one thing I want to address: The changed `template.txt` format for mods.
Previously, the "player" column was something like "5 = available to human", and "1, 3, 4, 5, and 6 = available to AI". This is WELL DOCUMENTED [1], and is used this way in all Pumpkin code, old mods, new mods, etc. ([1] see http://developer.wz2100.net/wiki/TxtEditing#Templates.txt ) In the changed code, the "player" column is simplified to "0 = available to human" and "every template = available to AI". Well, the main problem is, this drops compatibility with old mods. I was originally going to restore compatibility by changing it to "0 or 5 = available to human", but I discovered that the old `template.txt` files use "0" to mean "not available to humans", so it's still not enough to restore backwards-compatibility. I'm proposing another change that restores backwards-compatibility: rename the column to "available_to_humans", and have "5" or "YES" be available to humans (and every template is available to AIs). Then, for new `template.txt`, the column would always be either "YES" or "NO". There is precedent for the names: `weapons.txt` and many other mod files use "YES" and "NO" in columns. There is very little potential for bugs in this change, and is quite a minor change compared to, say, rewriting the template code, and I believe it is worth it, for backwards compatibility. Thoughts? -Zarel _______________________________________________ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
