Error after running script for some time

[iseahound]

Error after running script for some time

[iseahound]

Fixed by af2bca4 44a25c8 and 2fbc4fe.

[iseahound]

Is there a typo in your copy of ImagePut?

Line 1362 should read GlobalLock, not GlobalAlloc.

See:

ImagePut/ImagePut.ahk

Line 1362 in 4479b84

pData := DllCall("GlobalLock", "ptr", hData, "ptr")

[jpuerj]

Sorry, that's a typo in the topic, the file is correct! Going to edit it!

[iseahound]

Can you check for a memory leak? It looks like one to me. See how much memory AutoHotkey is using in task manager - it should be stable.

Also can you post the full code? Are you deleting 'img'?

[iseahound]

Also,

ImagePutBuffer("window")

is ambiguous syntax try:

ImagePutBuffer({window: "MyTitle"})

to avoid the bug where a file name has the same name as a window

[jpuerj]

Can you check for a memory leak?

It increased 10MB in the last 3 minutes, I'm going to keep an eye on it. I had discarded this possibility earlier because when I got a leak in the past it would crash very early. EDIT: look's like a leak, it increased 30MB since writting this post.

Are you deleting 'img'?

I'm not deleting image. I thought it was not needed because it's inside a class method.

Also can you post the full code?

Sure: script.zip

I'm going to try deleting 'img' and every related object. I just need img := "", right?

Thanks!

[iseahound]

Well this doesn't look like an ImagePut error.

Try to improve your memory management:

fn(img) {
    filepath := "something.png"
    static search := Imageputbuffer(filepath)
    img.imagesearch(search)
}

Make the search images static so you don't create a million of them. Then I see you're storing the screen capture in an array, you'll need to do arr[123] := "" to delete them. (Don't bother deleting static images they will be cleaned on script exit)

You probably didn't notice, but it will have to hit a few gigs of ram until it crashes like above.

[iseahound]

Add documentation to delete the buffers

Also, add an update() method to Imageputbuffer (for screen capture only)?

[jpuerj]

The problems seems to be with ImageSearch. Whenever the function is called the memory use increase a bit. I tested on this hotkey:

F1::
{
static img := ImagePutBuffer("Code")
static search := ImagePutBuffer("images/waypoint_1.png")
loop {
xy := img.ImageSearch(search)
Sleep(100)
}
}

The increase is very small... About 1MB each minute... I think that's why it took more than an hour to crash on my script.