r7313 | skill names can contain braces

[Technology]

Which means we cannot use the following notation for these skills:

Code: Select all

skillname (HANDLE)

Which we currently do.

[kLabMouse]

MM. let's filter all Skill names for "(" + ")" and replace to "[" + "]"
So Skill names will not make our system crash.

[Technology]

MM. let's filter all Skill names for "(" + ")" and replace to "[" + "]"
So Skill names will not make our system crash.

i suggest we adapt our system, i for one want to be able to "drag and drop" (as a figure of speech) in skillnametable.txt from the .grf
Why do we need this notation anyways?

btw, i think the same problem applies to statuses?

[kLabMouse]

Yes. the same.
As for me. Changing () -> [] isn't bad at all.
but.. most users use Skill name, or HANDLE. but not two of them in the same time.

[EternalHarvest]

"skillname" in this notation is just a random text (now), and this random text can be put in comments if needed instead.

[Technology]

"skillname" in this notation is just a random text (now), and this random text can be put in comments if needed instead.

yes, i think the same about it.

[kLabMouse]

OK.So we disable "Skill_Name (HANDLE)" as that makes troubles, and it's nonsense to use it.
only left "Skill_Name" or "HANDLE".

Same go for Statuses.