[NSIS-Bugs] [ nsis-Bugs-3612983 ] Problem with LogicLib LoopUntil

[SourceForge.net <no...@so...>]

Bugs item #3612983, was opened at 2013-05-09 10:59
Message generated for change (Tracker Item Submitted) made by digital56k2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373085&aid=3612983&group_id=22049

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Scripting
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Alastair (digital56k2)
Assigned to: Nobody/Anonymous (nobody)
Summary: Problem with LogicLib LoopUntil

Initial Comment:
Using ANSI NSIS 2.46, writing a script to populate a combobox with strings from an INI file, I have found that LogicLib isn't handling LoopUntil properly.

Here's a snippet:

Push $0 ; Index
Push $1 ; Value
ClearErrors
StrCpy $0 0
${Do}
	IntOp $0 $0 + 1
	ReadINIStr $1 "$EXEDIR\Strings.ini" "Strings" "String$0"
	${IfNot} ${Errors}
		${NSD_CB_AddString} $MyComboBox "$1"
	${Else}
		MessageBox MB_OK " Error for index $0"
	${EndIf}
${LoopUntil} ${Errors}
Pop $1
Pop $0

Now, when you run that you won't have the INI file and the error flag will be set immediately for Index 1, which is fine for testing. You can comment out the line using ${NSD_CB_AddString}, it doesn't affect the symptoms of this issue. What you'll see is the message box pop up because an error was detected for index 1, but then the outer loop will continue indefinitely.

If you remove the inner ${IfNot}, then the problem magically goes away:

Push $0 ; Index
Push $1 ; Value
ClearErrors
StrCpy $0 0
${Do}
	IntOp $0 $0 + 1
	ReadINIStr $1 "$EXEDIR\Strings.ini" "Strings" "String$0"
${LoopUntil} ${Errors}
Pop $1
Pop $0

Now the loop breaks properly, and you're no longer stuck in an infinite loop.

There is an example of using Do, LoopUntil Errors in the Examples folder, and the syntax is similar. Nothing in LogicLib.nsh seems to call ClearErrors directly.

What gives?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373085&aid=3612983&group_id=22049