Skip to content
This repository has been archived by the owner on Jan 25, 2018. It is now read-only.

Fix crashes related to large background bitmap #597

Conversation

eighthave
Copy link
Member

load background bitmaps in a custom AsyncTask

In order to support devices that do not have tons of RAM, dynamically size
the background bitmap when loading so only what is needed is stored in RAM.
Also, there is a much stricker limit of memory allocation when operating in
the main UI thread.  More info here:

https://developer.android.com/training/displaying-bitmaps/load-bitmap.html
https://developer.android.com/training/displaying-bitmaps/process-bitmap.html

fixes #4277 https://dev.guardianproject.info/issues/4277

In order to support devices that do not have tons of RAM, dynamically size
the background bitmap when loading so only what is needed is stored in RAM.
Also, there is a much stricker limit of memory allocation when operating in
the main UI thread.  More info here:

https://developer.android.com/training/displaying-bitmaps/load-bitmap.html
https://developer.android.com/training/displaying-bitmaps/process-bitmap.html

fixes #4277 https://dev.guardianproject.info/issues/4277
@n8fr8
Copy link
Member

n8fr8 commented Dec 16, 2014

Thanks for putting up with my desire for "decoration"

@n8fr8 n8fr8 merged commit 43e213a into guardianproject:master Dec 16, 2014
@eighthave eighthave deleted the fix-crashes-related-to-large-background-bitmap branch December 17, 2014 12:04
@eighthave
Copy link
Member Author

I also like pretty apps and think it is important, I just think we need to
prioritize and get a really stable release out. The OTR stuff is still pretty
problematic, for example.

@n8fr8
Copy link
Member

n8fr8 commented Dec 17, 2014

That's why we're doing incremental releases twice a month now, slowly but surely tracking all these things down.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants