use of keywords in MessageBox string

[asmo]

I don't know if the TES3 engine prohibited the use of keywords in a string.

statement

Code: Select all

MessageBox "OnActivate"

CS script check complains

Code: Select all

Error | (line) 85 | Unexpected keyword ("OnActivate")

the game engine

Code: Select all

error line 85, column 24 ("OnActivate")
    Unexpected keyword
compiling failed: asmo_conmer_chest_script

[Zini]

It just never ends. And this time I really have no idea how to fix this. Any approach I can think of would break other features. Sorry, but I think this is the first case where I have to declare that we won't fix it. At least not before OpenMW 1.1.

[asmo]

Okay

https://bugs.openmw.org/issues/2991
w/ priority "low" and target "openmw-future"

[densm27]

Just wondering how could English version of Morrowind work with that? There is no single script MessageBox'ing words like "begin" or "end" or "else"?
upd:
oh, I see, everything is OK if keyword is not alone in that string

[asmo]

Just wondering how could English version of Morrowind work with that? There is no single script MessageBox'ing words like "begin" or "end" or "else"?
upd:
oh, I see, everything is OK if keyword is not alone in that string

https://bugs.openmw.org/issues/2991#note-2
Looks like this was no problem in the TES3-engine.

[Zini]

Yeah. Our script engine is actually supposed to be a superset of the original one. Some of the original limitations were mind bending (showing that the developers had no idea how a proper compiler works) and replicating all that crap would have taken much longer than the more capable implementation we have now.
However when I originally wrote the scripting engine, I wasn't aware of some of the idiosyncrasies of the original, which has caused problems like this.

[asmo]

It's not that much of a problem, shipped scripts do not seem to contain a keyword only (makes sense).
As densm27 pointed out it's easy to avoid like this: MessageBox "_OnActive_". I think most of the keywords are used for debug statements, so no big deal. You just have to know that...

IMO Lowest priority, paaaast v1.0