commit | fb062c63355f65e6df9e99cbf4cce49cee2900d1 | [log] [tgz] |
---|---|---|
author | Tony Wickham <twickham@google.com> | Fri Oct 16 10:12:23 2015 -0700 |
committer | Tony Wickham <twickham@google.com> | Fri Oct 16 10:12:23 2015 -0700 |
tree | b0a73d272527cc1e54af67b743ba9d85241888a0 | |
parent | a8709ddbbf6338d71625ee1eef96e42bee042b31 [diff] |
Close some Cursors and call super.onFinishInflate(). Android Studio gave these warnings, and they seem like good ideas. Change-Id: I9a4d3997fd18e86e007e4511e1ac9e8a75509265
diff --git a/src/com/android/launcher3/PageIndicatorMarker.java b/src/com/android/launcher3/PageIndicatorMarker.java index f012db7..7bf21dd 100644 --- a/src/com/android/launcher3/PageIndicatorMarker.java +++ b/src/com/android/launcher3/PageIndicatorMarker.java
@@ -45,6 +45,7 @@ } protected void onFinishInflate() { + super.onFinishInflate(); mActiveMarker = (ImageView) findViewById(R.id.active); mInactiveMarker = (ImageView) findViewById(R.id.inactive); }
diff --git a/src/com/android/launcher3/model/MigrateFromRestoreTask.java b/src/com/android/launcher3/model/MigrateFromRestoreTask.java index 6a529f6..ba33b02 100644 --- a/src/com/android/launcher3/model/MigrateFromRestoreTask.java +++ b/src/com/android/launcher3/model/MigrateFromRestoreTask.java
@@ -632,9 +632,9 @@ mEntryToRemove.add(entry.id); continue; } - entries.add(entry); } + c.close(); return entries; } @@ -655,7 +655,7 @@ mEntryToRemove.add(c.getLong(0)); } } - + c.close(); return total; }