summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Yang Ni <yangni@google.com> 2016-04-19 18:02:01 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2016-04-19 18:02:01 +0000
commit1d0b9e2f46b680c4530dd89d5331d3a482bb21b9 (patch)
treed23df0e7a0e1c0a8ea57072e0b575380aa53c298
parente05bdb15e519fb0d7575d33cb08ff9e17b117568 (diff)
parente04e5d7cc0a8a1b2792448c885aed9f360e34b14 (diff)
Merge "Fixed ScriptGroup finalizer for old API"
-rw-r--r--rs/java/android/renderscript/ScriptGroup.java6
1 files changed, 5 insertions, 1 deletions
diff --git a/rs/java/android/renderscript/ScriptGroup.java b/rs/java/android/renderscript/ScriptGroup.java
index 35ae8b406e45..0d10c6d9dc50 100644
--- a/rs/java/android/renderscript/ScriptGroup.java
+++ b/rs/java/android/renderscript/ScriptGroup.java
@@ -1074,7 +1074,11 @@ public final class ScriptGroup extends BaseObj {
protected void finalize() throws Throwable {
// Clear out the list mClosures to avoid double-destroying the closures,
// in case their finalizers race ahead.
- mClosures.clear();
+ if (mClosures != null) {
+ // ScriptGroup created using the old Builder class does not
+ // initialize the field mClosures
+ mClosures.clear();
+ }
super.finalize();
}
}