Trouble with While/case, controls in an array (sliders)

[Werty]

I'm getting a constant 8-12% cpu load using this while loop, despite the Sleep()...

While 1
    
    $msg = GUIGetMsg()
        Switch $msg
            Case -3
                On_Exit()
        
            Case Else
                For $loop = 0 To 37
                    If $msg = $morphvalue[$loop] Then
                        _GUICtrlSlider_SetPos($morphhandle[$loop], GUICtrlRead($morphvalue[$loop]))
                        ControlSend("","",$morphhandle[$loop],"{UP}")
                        EndIf
                Next
            
        EndSwitch
        $msg = ""
    Sleep(10)
WEnd

It's an array of sliders in my GUI that controls sliders in another program.

It works as it should, but I'm getting cpu load eventhough I have the sleep() in there, removing the sleep() hogs the cpu even more obviously.

Edited August 18, 2011 by Werty

[PsaltyDS]

The Sleep() isn't required in the While/WEnd because GuiGetMsg() automatically imposes a delay when called in a loop. Your For/Next loop should only run if a $msg was received. Once the For/Next loop finds a match, it should exit.

While 1
    $msg = GUIGetMsg()
    If $msg Then
        Switch $msg
            Case -3
                On_Exit()
            Case Else
                For $loop = 0 To 37
                    If $msg = $morphvalue[$loop] Then
                        _GUICtrlSlider_SetPos($morphhandle[$loop], GUICtrlRead($morphvalue[$loop]))
                        ControlSend("", "", $morphhandle[$loop], "{UP}")
                        ExitLoop
                    EndIf
                Next
        EndSwitch
    EndIf
WEnd

[JohnOne]

I think its probably not that high if its a single core processor.

You could try adding a sleep to your For Next loop because thats what is doing the work.

A sleep is already added internally with GUIGetMsg()

[Werty]

The Sleep() isn't required in the While/WEnd because GuiGetMsg() automatically imposes a delay when called in a loop. Your For/Next loop should only run if a $msg was received. Once the For/Next loop finds a match, it should exit.

While 1
    $msg = GUIGetMsg()
    If $msg Then
        Switch $msg
            Case -3
                On_Exit()
            Case Else
                For $loop = 0 To 37
                    If $msg = $morphvalue[$loop] Then
                        _GUICtrlSlider_SetPos($morphhandle[$loop], GUICtrlRead($morphvalue[$loop]))
                        ControlSend("", "", $morphhandle[$loop], "{UP}")
                        ExitLoop
                    EndIf
                Next
        EndSwitch
    EndIf
WEnd

Thanks, that did it, back at 0% idle

I think its probably not that high if its a single core processor.

Quadcore, so it was eating away half a core :/