1.1.27 crashes Topic is solved

[guest3456]

running the Example.ahk unit tests from the /doc/ subfolder of this repo:

https://github.com/Uberi/Yunit

windows crash on v1.1.27
works successfully on 1.1.26.01

can narrow it down just to the GUI output by changing the top lines of Example.ahk to this:

Code: Select all

#Include ..\Yunit.ahk
#Include ..\Window.ahk

Yunit.Use(YunitWindow).Test(NumberTestSuite, StringTestSuite)

[lexikos]

You can add #NoEnv as a workaround for now.

[guest3456]

You can add #NoEnv as a workaround for now.

thanks yeah adding NoEnv to YUnit.ahk fixes it in v1. fyi, it still crashes on v2 Yunit branch w/ a82 too since NoEnv isn't availalble there

[lexikos]

No, it crashes for a different reason.

The first bug occurs if you use pre-increment/decrement with a potential environment variable, such as ++myvar. It does not occur in v2, which handles variables the same as if #NoEnv was in effect.

The second bug occurs if you use throw, because of an erroneous line that was added by automatic merge with v1. The heap is corrupted by throw, but you may not see the effect immediately.

There is a third bug relating to the first one which affects v1 and v2. If you use ++MyClass.x with #Warn, you will get a warning because ++ only looks to the next token for its target variable. Fixing this means delaying the check to a later stage, which incidentally also fixes the first bug.

[Nextron]

The first bug occurs if you use pre-increment/decrement with a potential environment variable, such as ++myvar.

You can add #NoEnv as a workaround for now.

Thanks, good to know! I did identify the pre-increment as a culprit (and was in the process of replacing them in my script), but wasn't able to reliably reproduce crashes because I didn't made the link to #NoEnv yet.

[lexikos]

Fixed in v1.1.27.01.