This is a little confusing then, though. The animation thread dies as expected when I stop the application using the back arrow, but does not die (without being explicitly killed) when I flip the orientation. So some extra cleaning up must be happening for one that isn't happening for the other.
Also, any ideas on the lag? On Oct 12, 11:00 am, hackbod <[EMAIL PROTECTED]> wrote: > onCreate -is- called when the activity is starting fresh. When an > orientation switch happens, the old activity is removed through > onPause()->onStop()->onDestroy(), and a new instance created in the > new orientation. > > On Oct 11, 10:15 pm, songs <[EMAIL PROTECTED]> wrote: > > > Tracked the leak down to my animation thread which was getting > > recreated on every onCreate. From the lifecycle docs, I had thought > > that onCreate only got called when the activity was starting fresh and > > any old instances had died (and killed everything associated with > > it). Since this apparently isn't always the case, I'm now managing > > this thread more actively. > > > This mostly solved the issue except for two things - I can still get > > the memory issue to happen (I just have to try harder), and after > > changing the orientation it takes about 2-3 seconds before the app > > will respond to touch events. From watching the heap using DDMS, > > there's a good a couple of seconds between the thread dying and its > > resources getting cleaned up. Because of this lag, if I quickly > > switch back and forth between orientations I can get the memory error > > to happen. > > > These remaining issues are kind of on outside of expected behavior so > > I'm not too worried about things now, but there's gotta be something > > I'm missing that would help button this up a little tighter. > > > - Steve > > > On Oct 10, 1:53 am, songs <[EMAIL PROTECTED]> wrote: > > > > Interesting. Thanks for the lead. I'll look into this an report back > > > either way. > > > > On Oct 10, 1:48 am, "Romain Guy" <[EMAIL PROTECTED]> wrote: > > > > > Then you should check carefully where exactly you pass your Activity > > > > (or Context) to other classes/APIs, etc. The Context is used by many > > > > different classes in the Android framework and if you somehow "leak" > > > > the Context, you leak all the views, images, etc. Your issue is very > > > > similar to several issues we fixed over the past few months: on each > > > > rotation the Context leaks and everything leaks with it. You can use > > > > DDMS to check how the heap is growing after each rotation, it should > > > > tell you whether this is the problem or not. > > > > > On Fri, Oct 10, 2008 at 1:44 AM, songs <[EMAIL PROTECTED]> wrote: > > > > > > Nope, the only static variables I have are constants. I do keep an > > > > > array of Drawable in the view, but that's an instance variable. > > > > > > On Oct 10, 1:14 am, "Romain Guy" <[EMAIL PROTECTED]> wrote: > > > > >> Looks like you're leaking memory somewhere. Do you keep a static > > > > >> cache > > > > >> somewhere in your app? > > > > > >> On Thu, Oct 9, 2008 at 10:02 PM, songs <[EMAIL PROTECTED]> wrote: > > > > > >> > Hi, > > > > > >> > I think that either I've stumbled onto a bug or I'm missing > > > > >> > something > > > > >> > on how to manage my resources. I have an activity with a Java > > > > >> > based > > > > >> > view that contains 30 or so small images (~2k each .png) that are > > > > >> > loaded into an array when the view is created. In "normal" > > > > >> > operation, > > > > >> > everything runs fine. When I flip the orientation between > > > > >> > landscape > > > > >> > and portrait 3-4 times, I get the following: > > > > > >> > E/AndroidRuntime( 472): java.lang.OutOfMemoryError: bitmap size > > > > >> > exceeds VM budget > > > > >> > E/AndroidRuntime( 472): at > > > > >> > android.graphics.BitmapFactory.nativeDecodeAsset(Native Method) > > > > >> > E/AndroidRuntime( 472): at > > > > >> > android.graphics.BitmapFactory.decodeStream(BitmapFactory.java:290) > > > > >> > E/AndroidRuntime( 472): at > > > > >> > android.graphics.drawable.Drawable.createFromStream(Drawable.java:635) > > > > >> > E/AndroidRuntime( 472): at > > > > >> > android.content.res.Resources.loadDrawable(Resources.java:1440) > > > > >> > E/AndroidRuntime( 472): at > > > > >> > android.content.res.Resources.getDrawable(Resources.java:498) > > > > > >> > When the orientation gets changed, the activity seems to go through > > > > >> > the entire lifecycle (pause, stop, then create again). A new > > > > >> > instance > > > > >> > of the activity's view is created in the onCreate method and it > > > > >> > seems > > > > >> > that the pause/stop steps in the lifecycle don't clean up the old > > > > >> > view. I've tried explicitly setting the reference to the view to > > > > >> > null > > > > >> > in onPause and doing a System.gc(), but that doesn't seem to help. > > > > >> > I've also tried setting the requested orientation to portrait in > > > > >> > hopes > > > > >> > that the activity would then ignore orientation changes, but that > > > > >> > didn't work either. GC messages come up after every orientation > > > > >> > change so one would think that the garbage collector was doing its > > > > >> > job. > > > > > >> > I could just leave this be and assume people aren't going to be > > > > >> > randomly flipping their phone open and closed while they're using > > > > >> > my > > > > >> > app, but that seems pretty sloppy. I'd like to know what's going > > > > >> > on > > > > >> > here and fix it. > > > > > >> > Any ideas? > > > > > >> > Thanks, > > > > >> > Steve > > > > > >> -- > > > > >> Romain Guywww.curious-creature.org > > > > > -- > > > > Romain Guywww.curious-creature.org --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Android Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/android-developers?hl=en -~----------~----~----~----~------~----~------~--~---
