wurstscript / WurstScript

Programming language and toolkit to create Warcraft III Maps
https://wurstlang.org
Apache License 2.0
226 stars 28 forks source link

Use original object ID if new object ID is 0 #985

Closed jmrico01 closed 3 years ago

jmrico01 commented 3 years ago

Simplest fix for #984, based on these few comments. If this is all that's needed, great! If you had other plans for this fix or wanted to include some more changes, please feel free to close this PR.

Also changed Map to LinkedHashMap per @peq's comment, to keep the order deterministic.

Frotty commented 3 years ago

Nice. Could only become a problem if someone assigns 0 als newId but I guess that's fine. 👍