Fix variable names with long numbers

[TPGamesNL]

Description

Changes the variable name comparator to not rely on long parsing, instead comparing digits manually.

Output from the test script in the first linked issue:

Other test code:

command /test2:
	trigger:
		loop 300 times:
			set {_i} to -1 + (loop-value) / 100
			set {_l::%{_i}%} to "a"
		loop {_l::*}:
			message loop-index

Output: https://pastebin.com/htAPMQXg

---

Target Minecraft Versions: any
Requirements: none
Related Issues: #4729, #3929 (latter non-fixing)

[APickledWalrus]

As long as this has been tested, it should be fine. A test should be added though if possible.

I also have to ask, why are variable names compared to this extent? Is there a reason to know more than whether or not the names are the same?

[TPGamesNL]

I also have to ask, why are variable names compared to this extent? Is there a reason to know more than whether or not the names are the same?

If we want to loop a list variable with number indices, we want to make sure it loops from 1-n, not an arbitrary order. Same for non-number indices, we want them to loop in alphabetical order, but also if we have sth like

• {test::abc-1}
• {test::abc-2}
• {test::abc-30}
  this should loop in the right order of numbers too