Ideally the Cities & Knights (CK) expansion rules will be under a
"Cities & Knights" heading, and tabbed over. These functions allow for
that to happen.
Hello,
Finally I've found the time to review the patch.
It looks OK, but I find the new function complex: it has so many arguments. Are all arguments necessary? I think only a left indent should be sufficient to fill in the need.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hello,
Finally I've found the time to review the patch.
It looks OK, but I find the new function complex: it has so many arguments. Are all arguments necessary? I think only a left indent should be sufficient to fill in the need.