Hi,

There were some patches committed (1.9) that make 'side=1' no longer a
default value for some of WML direct action tags (i.e.
https://gna.org/patch/?1883 - change for [gold] tag)

The intent was to make those tags do the action 'for all sides' if side=
is missing (same behavior as for unit and location filters - 'empty
matches all')

However, this led us to the situation where some tags still use side=1
default and some don't. this is not good, and I want to fix it, one way
or another.

An additional problem is, that side=1 default was used in a number of
UMC and it not easily WMLLINTable.

Possible ways to deal with the situation :

1. ignore it, let the mess with 'some tags having side=1 default and
some tags not having it' continue.
2. revert the patches, returing to 'side=1 is default for all direct
action tags' status quo.
3. modify all direct action tags to have "side=1 should be explicitly
set, if it's not set it's 'work for all sides'" behavior.
4. modify all direct action tags to have "side=1 should be explicitly
set, if it's not set it's an error" behavior.
5. modify all direct action tags to have "side=1 should be explicitly
set, if it's not set it's an error" behavior, plus add support for
[filter_side] tag (a standard side filter, a standard way to filter for
sides like 'all enemies of side 1') for all the direct action WML tags.

I personally think that (5) is the way to go, but I'm only strongly
opposed to (1).

What do you think ?


P.S.: Note that (2),(4),(5) can be done with full WMLLINT support. and
(1) and (3) can't be easily checked by WMLLINT.


Cheers,
Iurii Chernyi (Crab)

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Wesnoth-dev mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-dev

Reply via email to