Just some bugs introduced with my PR.
GZDoom spits out non-sensical errors if you attempt to initialize DynArrays, and I forgot to initialize the register for the array's stack offset.
About the DynArray bug, it'll complain about something being a non-value or something.
Right now I'm actually working on adding support for dynamic array initialization, and I've already fixed the stack offset, so I'm just posting this here so people can remind me if I get side-tracked or forget about it. (and also so I can link the PR here once it's done)
Array initialization bugs
Moderator: GZDoom Developers
Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
- phantombeta
- Posts: 2088
- Joined: Thu May 02, 2013 1:27 am
- Operating System Version (Optional): Windows 10
- Graphics Processor: nVidia with Vulkan support
- Location: Brazil
- phantombeta
- Posts: 2088
- Joined: Thu May 02, 2013 1:27 am
- Operating System Version (Optional): Windows 10
- Graphics Processor: nVidia with Vulkan support
- Location: Brazil
Re: Array initialization bugs
PR here. This adds support for dynamic arrays, fixes some nasty bugs and adds an error message when attempting to initialize multi-dimensional arrays.
I'm somewhat ashamed of how many mistakes I made in the original PR for the array initializers.
I'm somewhat ashamed of how many mistakes I made in the original PR for the array initializers.
Re: Array initialization bugs
Don't be. You're willing to fix it, and that counts a lot.phantombeta wrote:I'm somewhat ashamed of how many mistakes I made in the original PR for the array initializers.
- phantombeta
- Posts: 2088
- Joined: Thu May 02, 2013 1:27 am
- Operating System Version (Optional): Windows 10
- Graphics Processor: nVidia with Vulkan support
- Location: Brazil
Re: Array initialization bugs
This can be closed.