3.5.3-dev+r2439: persian swordmans' guard_attacking model cannot be loaded

[tomreyn]

While playing a single player game, indians (me) against persian (CPU) I got this:

Code: [Select]

[2011-07-05 15:22:51] *ERROR* In [/home/user1/SCM/megaglest-trunk/source/glest_game/types/skill_type.cpp::load Line 167] WARNING CANNOT LOAD MODEL [/home/user1/SCM/megaglest-trunk/mk/linux//techs/megapack/factions/persian/units/swordman/models/guard_attacking.g3d] for parentLoader [/home/user1/SCM/megaglest-trunk/mk/linux//techs/megapack/factions/persian/units/swordman/swordman.xml]

It should probably say 'swordman_attacking.g3d' only, not 'guard_attacking.g3d', in techs/megapack/factions/persian/units/swordman/swordman.xml?

[softcoder]

Now that we support multiple animations per skill, any animation files that do not exist will give a warning on the console. There are some xml's which contain multiple entries for animations where some of the files do not exist (such as the one you mentioned) and this should be cleaned up before release.

[tomreyn]

It's good that this now getting tested for, too.
However, should it not either say "warning" or "error", not both? I assume this is a warning, since it doesn't prevent the game from running, and one the game is able to handle fine.

[Omega]

I assume this is a warning, since it doesn't prevent the game from running, and one the game is able to handle fine.

Provided that at least one of the models can be loaded, yeah.

[softcoder]

Indeed its a warning if at least 1 model per skill is loadable, all other missing files are warnings