Comment 30 for bug 536076

Revision history for this message
Nicolai Hähnle (nha) wrote :

I'm in favor of keeping it this way.

The reason is that it avoids confusion with "ghost files". If you look at Chuck's commits, there are several ones where he changes the FPS at which things are rendered simply because some people end up with different versions of Widelands data files in the search paths. This causes the frame loading code to load files that are not supposed to be there (but end up in the search path by accident). This is very counterintuitive to people.

To avoid similar problems happening with playercolor files, I would prefer to keep the flag. (In fact, given the above, it would make sense to *add* an additional key/value pair indicating the number of frames in an animation. Or to radically change the way the file loading works. Oh well.)