Comment 13 for bug 1121982

Revision history for this message
Kiscsirke (csirkeee) wrote :

There were some further erroneous conf files, I corrected them in rev #6536.

I think the difference isn't Windows/Linux, but that you built a Release version, and that does less checks? If so, try compiling a debug build and testing with that. (The compile.sh script asks if you want a debug or release build.)

I don't know what your current process for editing these is, but I guess it could be more automatized, from seeing the number of potential mistakes?

Also 2 other comments on the branch:
 - Is there any reason for 'dimensions' and 'base_offset' to be a different format from 'hotspot'? (They use commas instead of spaces.) Being the same would be more logical to me. I would think it could also use the same code?
 - I think we should set some conventions on the image file naming. Right now there are folders where it's "idle_xx" and so on, and there are folders where the image files are named after the object too. With the conversion in this branch, some were converted from the former to the latter. I would actually prefer the former, it makes it easy to search for files "idle*" in all those folders, and the folder name already describes the object anyway. But again, setting some convention and sticking with it would be better in either case.