Actually, I believe I've found the problem with my original code.

The last use of the 'imageRep2' is on the line prior to 'A)'. Now, if the Java garbage collector hits in between 'A)' and 'B)', imageRep2 will get collected, causing the retain count to go down. And consequently, this will/may cause the memory area pointed to by 'pointer' to get freed, even though we still have a pointer to it. Both, adding an additional retain/release for imageRep2 seems to fix the crashing, as well as adding an additional reference to it in the end of the method, e.g. 'size = imageRep2.size()' to the second to last line.

Sami

--

private Screenshot _getScreenshot(int windowId) { NSRect nullRect = new NSRect(new NSPoint(Double.POSITIVE_INFINITY, Double.POSITIVE_INFINITY), new NSSize(0, 0)); ID imageRef = QuartzLibrary.INSTANCE.CGWindowListCreateImage(nullRect, QuartzLibrary.kCGWindowListOptionIncludingWindow, windowId, QuartzLibrary.kCGWindowImageBoundsIgnoreFraming); NSBitmapImageRep imageRep = NSBitmapImageRep.CLASS.alloc().initWithCGImage(imageRef); Foundation.cfRelease(imageRef); NSSize size = imageRep2.size(); Pointer pointer = imageRep2.bitmapData(); // A) if(pointer == Pointer.NULL) return null;

int width = size.width.intValue(); int height = size.height.intValue(); int[] data = pointer.getIntArray(0, width * height); // B) return new Screenshot(width, height, data); }

On Fri, Nov 6, 2009 at 12:00 PM, Paul Loy <pa...@keteracel.com> wrote: