Review my code

[vaibhavs]

Hi,

I have a simple apps which prevents PC users from deleting files & Folders.

I envoke this apps via registry on bootup.

HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\Run

Created new STRING VALUES [DoDel]

DoDel = c:\windows\DoDel.exe

Opt("TrayIconHide", 1)
while 1
        WinKill("Confirm File Delete", "")
        WinKill("Confirm Multiple File Delete", "")
        WinKill("Confirm Folder Delete", "")
WEnd

Many times, I see that this apps uses 80-90% of CPU resources.

I have not idea why?

Can someone share their thoughts on the possibility or some way to diagnose it.

Vai

[DW1]

Put a sleep in there, you are maxing out the processor by completing that loop as fast as the processor allows

EDIT: Sleep(10) would do the trick

Edited February 12, 2008 by danwilli

[Xenobiologist]

Hi,

look at sleep

Mega

[GEOSoft]

Hi,

I have a simple apps which prevents PC users from deleting files & Folders.

I envoke this apps via registry on bootup.

HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\Run

Created new STRING VALUES [DoDel]

DoDel = c:\windows\DoDel.exe

Opt("TrayIconHide", 1)
while 1
        WinKill("Confirm File Delete", "")
        WinKill("Confirm Multiple File Delete", "")
        WinKill("Confirm Folder Delete", "")
WEnd

Many times, I see that this apps uses 80-90% of CPU resources.

I have not idea why?

Can someone share their thoughts on the possibility or some way to diagnose it.

Vai

Opt("TrayIconHide", 1)
while 1
   If WinExists("Confirm File Delete", "") Then WinKill("Confirm File Delete", "")
   If WinExists("Confirm Multiple File Delete", "") Then WinKill("Confirm Multiple File Delete", "")
   If WinExists("Confirm Folder Delete", "") Then WinKill("Confirm Folder Delete", "")
   Sleep(10)
WEnd

[vaibhavs]

I will try the sleep(10).

Thx a ton to all.

Cheers,

Vai

[Squirrely1]

If you don't tell them how you exit the program, you can use Alt+Shift+F7 to exit this version:

Opt("TrayIconHide", 1)
HotKeySet("+!{F7}", "ExitProggy")
Sleep(280)
EmptyWorkingSet()
While 1
    If WinExists("Confirm File Delete") Then
        WinKill("Confirm File Delete")
    EndIf
    Sleep(4)
    If WinExists("Confirm Multiple File Delete") Then
        WinKill("Confirm Multiple File Delete", "")
    EndIf
    Sleep(4)
    If WinExists("Confirm Folder Delete") Then
        WinKill("Confirm Folder Delete")
    EndIf
    Sleep(11)
WEnd
Func EmptyWorkingSet()
;Puts as much of the script as possible onto the paging file
;Author: wOuter (mostly)
    Sleep(300)
    DllCall("psapi.dll", 'int', 'EmptyWorkingSet', 'long', -1)
    Sleep(1380)
EndFunc  ;==>EmptyWorkingSet
Func ExitProggy()
    Exit
EndFunc  ;==>ExitProggy

[James]

Hey this would be good if you could tell where the file delete was coming from e.g. Windows!