[MDS]Parsing brackets in .mds mesh entries stops processing

[JaXt0r]

Happy New Year, Luis! Let's rock and roll right from day 1. ;-)

Current behavior

If a registerMesh entry inside a .mds file contains round brackets/parenthesis, the parsing stops with this warning:
level=Warning, name=ModelScript, message=detected invalid use of KEYWORD ".ASC" in "Model" block. Ignoring rest of script.
Causing the whole file to stop being processed further and won't register animations (which are normally stored below).

Example: registerMesh ("Some_File Name(WithBrackets).ASC")

Reproduction

1. Use ZenKit version = 1.0.3.0
2. Create .mds file with this value

Model ("HuS")
{
	meshAndTree ("Hum_Body_Naked0.ASC" DONT_USE_MESH)
	registerMesh ("Some_File Name(WithBrackets).ASC")

	aniEnum
	{
		modelTag	("DEF_HIT_LIMB"		"zs_RightHand")
        ani			("s_Run"				1	"s_Run"			0.1	0.1	MI	"Hum_RunAmbient_M01.asc"	F	1	50)
    }
}

Expected behavior

Parsing should accept brackets >()< as they're put in double quotes and are valid file names on linux and windows.

[lmichaelis]

Problem is that we need to have a special case for baseline Gothic and Gothic 2 because with this new information all of the following is valid:

registerMesh ("Some_File Name(WithBrackets).ASC")
registerMesh ("Some_File Name.ASC")
registerMesh ("Some_File Name.ASC)
registerMesh ("Some_File Name.ASC

ZenKit/src/ModelScriptDsl.cc

Lines 81 to 98 in 0e3b3f6

	if (chr == '"') {
	chr = static_cast<unsigned char>(_m_buffer->read_char());
	_m_column += 1;
	while (chr != '"' && chr != '\n' && chr != ')') {
	_m_value.push_back(static_cast<char>(chr));
	chr = static_cast<unsigned char>(_m_buffer->read_char());
	_m_column += 1;
	}
	if (chr != '"') {
	WARN_SYNTAX("String not terminated");
	_m_buffer->seek(-1, Whence::CUR);
	_m_column -= 1;
	}
	return MdsToken::STRING;
	}