DH_ARRAYSET implementation is wrong

DH_ARRAYSET implementation is wrong

Alright, I've written a smaller, more concise and fool proof answer in the tracker since someone wasn't able to spot the issue.

[TarCV]

Yes, result of the call memset(1, 1, 32768) in HUMANBOT would be globalArray1 elements all set to 0x01010101. And I'm not sure if memset command is actually useful for a modder.
But probably there is nothing to do to fix that on botc side. At least other than making the compiler stable enough to be used for building Zandronum wad, and thus allowing to use bot sources instead of compiled bytecode.

Is a simple for loop sufficient in the VM? The str type will have the same size as an int, right?

[TarCV]

I'm not sure it's possible to put a string value into a global array in bot scripts at all. I can't find any DH_ code that pops a string stack value and puts it into a variable or an array.

In any case, I have submitted a PR.
https://bitbucket.org/Torr_Samaho/zandronum-stable/pull-requests/89/bots-dh_arrayset-implementation-is-wrong/diff

[TarCV]

Thank you. May be it'll even fix some weird behavior of the bots.

Are you referring to this?
https://zandronum.com/tracker/view.php?id=3774

Commit corrected and accepted.
https://bitbucket.org/Torr_Samaho/zandronum-stable/commits/a04a4f94737e5ad3aad6573227e146b76f037d95