In the process of adding the IGNOREPREREQ faction ability (what I had previously been calling CANBUILDUNIT, but with somewhat more options), I can make a few changes to the way faction files are parsed, to make them more accommodating of common errors. These are:
1. If something does not require a value, such as WORMPOLICE or TECHSTEAL, it will still treat the following entry as its "value", so something like WORMPOLICE, SOCIAL, ++RESEARCH would not give the desired +2 RESEARCH until it was changed to WORMPOLICE, 0, SOCIAL, ++RESEARCH. I can remove this, so that WORMPOLICE, SOCIAL, ++RESEARCH gives +2 RESEARCH...however, this will mean that WORMPOLICE, SOCIAL, SOCIAL, ++RESEARCH, which previously would have given +2 RESEARCH, now thinks it's asking for "SOCIAL, SOCIAL" and "++RESEARCH", which are meaningless, so it gives only WORMPOLICE. This would be no extra effort at all.
2. If something is meaningless (e.g. SOICAL) it is currently ignored. I think I can have it give an error message instead with only minor extra effort. (If #1 is used, the unnecessary number after WORMPOLICE or its like would not give an error message, but any non-number things would.) This will help catch typos, but may cause certain faction files with harmless mistakes to not work until the mistake is fixed.
So...which changes would you like to see, which would you like to not have, and for which do you not care?
