From 47faa1f69edd3772de74a6cd1f05bdb06dcabe7b Mon Sep 17 00:00:00 2001 From: hyb1996 <946994919@qq.com> Date: Fri, 30 Mar 2018 12:48:09 +0800 Subject: [PATCH] fix(images): it seems that memory leak of `images` has been fixed closes #323 --- .../com/stardust/autojs/core/image/ImageWrapper.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/autojs/src/main/java/com/stardust/autojs/core/image/ImageWrapper.java b/autojs/src/main/java/com/stardust/autojs/core/image/ImageWrapper.java index 6751b38a..857b19a3 100644 --- a/autojs/src/main/java/com/stardust/autojs/core/image/ImageWrapper.java +++ b/autojs/src/main/java/com/stardust/autojs/core/image/ImageWrapper.java @@ -76,7 +76,7 @@ public class ImageWrapper { int rowPadding = plane.getRowStride() - pixelStride * image.getWidth(); Bitmap bitmap = Bitmap.createBitmap(image.getWidth() + rowPadding / pixelStride, image.getHeight(), Bitmap.Config.ARGB_8888); bitmap.copyPixelsFromBuffer(buffer); - if(rowPadding == 0){ + if (rowPadding == 0) { return bitmap; } return Bitmap.createBitmap(bitmap, 0, 0, image.getWidth(), image.getHeight()); @@ -133,4 +133,13 @@ public class ImageWrapper { } } + + @Override + protected void finalize() throws Throwable { + try { + recycle(); + } finally { + super.finalize(); + } + } }