TarCV / botc

Fork of Dusk's botc compiler *OPEN FOR ADOPTION*

Other

1 stars 0 forks source link

Zandronum reports stack size exceeded when Roam() is used #10

Closed ghost closed 4 years ago

ghost commented 4 years ago

This bug applies to tarcv-master only, don't know if it applies to upgrade.

TarCV commented 4 years ago

I think it's because botc doesn't drop unused return values from the stack. I'll try to fix it on weekend.

TarCV commented 4 years ago

@so-sleepy, seeing your like on that PR, I suppose it fixed the issue for you?

ghost commented 4 years ago

You guessed that right, I needed this fix to modify an annoying behavior of the bots, which was a bug, thank you so much. Would you mind if I began testing the upgrade branch and submitting issues?

TarCV commented 4 years ago

Btw, you might be also interested in DH_ARRAYSET bug. From my understanding of bot sources Carnevil's intent doesn't match DH_ARRAYSET implementation.

Help with testing that branch would be really nice, so no issues with that. Have you encountered any other issues with PR #7?

ghost commented 4 years ago

14 doesn't fix the upgrade branch, I have also commented in #7.