[r2089] GAMEINFO LOAD error.

[Enjay]

As discussed in the Wiki thread, there was an error when trying to use a GAMEINFO with this in it:

Code: Select all

LOAD = test1.wad, test2.wad

From my reading of the changelog, it looks as if there may have been an attempt to address this however, from an end user perspective, all that has happened is that the error message has changed:

Instead of saying in r2081:

Code: Select all

Script error, "GAMEINFO" line 1:
Expected '=' but got '.' instead.

it says

Code: Select all

Script error, "GAMEINFO" line 1:
Expected identifier but got '.' instead.

and just for completeness:

Code: Select all

LOAD = "test1.wad", "test2.wad"

works properly in both revisions.

[Graf Zahl]

I'm not going to change this. The 'official' syntax will be with quotes but it will silently accept unquoted single names.

To get this working some really ugly messing around with the parsed data would be necessary and it's just not worth it.

[Blzut3]

but it will silently accept unquoted single names.

Is there any reason for this? Considering that it was just added, wouldn't it seem logical to make the syntax as strict as possible to prevent future problems?

[Graf Zahl]

It's basically the same rule as for MAPINFO's new syntax. It also silently allows a few things that with a strict approach should fail. But since all of this is still highly experimental we'll see.

I'd first like to add a few more features to this lump to make it really useful (like specifying autoload and config sections in the INI, for example.)