recompiler icon indicating copy to clipboard operation
recompiler copied to clipboard

Spaces in folder names don't work right

Open twist84 opened this issue 8 years ago • 5 comments

If you have a game in D:\Games\Consoles\X360\Homebrew\Quake 3\ after Quake the rest of the folder name doesn't get parsed.

twist84 avatar Oct 21 '17 12:10 twist84

Is this in the GUI tools or via command line ?

rexdex avatar Oct 21 '17 12:10 rexdex

GUI tools, I haven't checked via command line

twist84 avatar Oct 21 '17 12:10 twist84

Just checked via command line and it's the same even with quotation around the path"D:\Games\Consoles\X360\Homebrew\Quake 3\q3_360.xex"

twist84 avatar Oct 21 '17 12:10 twist84

Yeah, that looks about right. CommandLine::Parse has this loop where we get the value of each property specified, and we do this by moving forward on the string, one character at a time, until we encounter a space. It's worth noting that argv actually gives us the whole path (with spaces, all in quotes), but we screw up when parsing it.

@rexdex I can take this if you want. Not sure if the GUI tools use the same API.

sumit0190 avatar Oct 21 '17 14:10 sumit0190

Yeah, go ahead. In general the space is a separator unless it's in the "", maybe the logic is flawed there.

rexdex avatar Oct 21 '17 15:10 rexdex