Geoff the Medio wrote:
bastion wrote:
Seems like it's getting interpreted somewhere without properly escaping the backslash, causing \0 to turn into a null character.
Ugh.
Yes, that was my first thought too. Fortunately it isn't as bad as it looks

Quote:
Well, I'm not going to try to fix that before a release. So as above, I suggest installing into a more-regular path like D:\Games\FreeOrion.
After looking into the matter I tend to disagree here: This bug is quite serious, I wonder why there haven't been more people running into this problem on windows. To illustrate this let's take your suggestion for a more-regular path above: "D:\Games\FreeOrion". That one would work. But let's assume there is some guy who decides to use this path instead (the difference is really minimal!): "D:\Games\freeorion". This would turn out as "D:\\Games\x0creeorion" - uh-oh. The cause for this is the way string literals are handled in python, bastion already provided the link to the
corresponding section in the python documentation in his post above (I won't quote the entire section here). As you can see there, there are quite some characters affected by this escape sequence parsing of string literals...
I don't think you want an unresolved issue of that kind in 0.4, do you?

Fortunately, going through this part of the python doc also gave me clues how to solve the problem. The code section in question is this one from PythonAI.cpp:
Code:
// tell Python the path in which to locate AI script file
std::string AI_path = (GetResourceDir() / "AI").string();
std::string path_command = "sys.path.append('" + AI_path + "')";
object ignored = exec(path_command.c_str(), s_main_namespace, s_main_namespace);
The path to the AI modules is inserted as a string literal into the command string passed to exec, ergo being treated as such by the interpreter. This isn't a problem on platforms that use "/" as the path delimiter, but becomes a big one on windows, where "\" is the path delimiter. However, python allows you to prefix string literals with "r", which causes the interpreter to skip the escape sequence parsing for the most part (details see link above). So putting a "r" before the string literal constructed with AI_path should solve the problem:
Code:
std::string path_command = "sys.path.append(r'" + AI_path + "')";
As I'm not able to build FO on the windows PCs at my disposal, I can't test my theory. However, I've been able to simulate the issue in the python shell on my old windows laptop, and the results look very promising. As the required change is a very small one, I think it's worth trying

And as this change shouldn't affect the functionality of this piece of python code on other platforms, it shouldn't even be required to split the section into platform specific code parts.
This is going to be the smallest patch to any piece of code I've come up and will come up with for the rest of my life. I mean, how I'm going to go below one character...?
