From 7fe0fcba480d301af8c629e4a37d11e5fbd5f8fc Mon Sep 17 00:00:00 2001 From: Samuel Tardieu Date: Wed, 8 Jan 2014 18:43:17 +0100 Subject: Remove onTerminate() in application Not only it is not advised to derive from `Application` as this is often deemed useless, but also the `onTerminate()` method is never called on real devices. Since it causes issues with our tests since it accesses ressources that appear to have been cleaned up already, such as the application context, it is best to remove it entirely. --- main/src/cgeo/geocaching/CgeoApplication.java | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'main/src/cgeo/geocaching/CgeoApplication.java') diff --git a/main/src/cgeo/geocaching/CgeoApplication.java b/main/src/cgeo/geocaching/CgeoApplication.java index 2500d10..f4cf6c9 100644 --- a/main/src/cgeo/geocaching/CgeoApplication.java +++ b/main/src/cgeo/geocaching/CgeoApplication.java @@ -45,16 +45,6 @@ public class CgeoApplication extends Application { DataStore.removeAllFromCache(); } - @Override - public void onTerminate() { - Log.d("Terminating c:geo…"); - - DataStore.clean(); - DataStore.closeDb(); - - super.onTerminate(); - } - /** * Move the database to/from external cgdata in a new thread, * showing a progress window -- cgit v1.1