Jump to content

Can't release memory


Recommended Posts

I have this command:

$ping = Run( @ComSpec & " /c ping www.google.com -t" , @SystemDir , @SW_HIDE , $STDOUT_CHILD )

and this exit function:

OnAutoItExitRegister( "_Esc" )
Func _Esc()
ProcessClose( $ping )
ProcessClose( "PING.exe" )
ProcessClose( "conhost.exe" )
Exit
EndFunc

but if i run the script and then run this command:

C:\Windows\System32\cmd.exe /q /c ping www.google.com -t

it opens another 2 "PING.exe" and "conhost.exe" processes and then the script doesn't know which of them to close

How can I release memory so that when I exit the script it also closes the processes that it opened?

[center]Sorry for my bad english. Trying my best :Dhttp://iulianonofrei.comhttp://www.last.fm/user/Revolt666 [/center]

Link to comment
Share on other sites

The Run command returns a PID (Process Identifier), ProcessClose takes either a program name or a PID, I'd use the PID from Run to close the one that I opened.

If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Link to comment
Share on other sites

Run the Ping program directly, there's no reason to use @Comspec before it. Then when the program closes it will be able to close the Ping program without worrying about trying to close cmd.exe first.

OnAutoItExitRegister("_Esc")
$Ping = Run("ping www.google.com -t", @SystemDir, @SW_HIDE, 8)
Sleep(3000) ; sleeps for 3 seconds, then the program closes
Func _Esc()
    ProcessClose($Ping)
    Exit
EndFunc   ;==>_Esc

If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...