[V2.beta.2] Loop Until do not support block Topic is solved

[aseiot]

Code: Select all

loop
{
	x := A_Index
	ToolTip(x)
}until A_Index = 100


MsgBox(x)

[TheArkive]

EDIT: My bad... budding out!

[iseahound]

Definitely not working fine.

Code: Select all

      loop {
         if (A_Index > 5)
            throw Error("☹")

         str := "happy" . A_Index
      } until InStr(str, "happy2")

EDIT: I feel like I'm responsible for breaking loop-until even more

[swagfag]

it works fine if u havent yet updated to the commit that made it no longer work fine: https://github.com/Lexikos/AutoHotkey_L/commit/85202cab71352026704ffb787b2fa8ae5afcc03e (...probably)

[thqby]

I think line 5247 should be changed to if (aActionType == ACT_UNTIL && (!mPendingRelatedLine || mPendingRelatedLine->mActionType != ACT_LOOP))

[lexikos]

@thqby That would defeat the purpose of the check, by allowing Until to "relate to" more than one Loop. It also fails to take into account For..Until.

The fix for this bug must also take into account Loop-x-Until where x is any nested statement other than a block (as this is currently not working either).

I think 6380b3b does it.