Skip to content
This repository has been archived by the owner on Sep 6, 2019. It is now read-only.

3.5.8 UI Crash #2095

Closed
an0n981 opened this issue Dec 24, 2014 · 30 comments
Closed

3.5.8 UI Crash #2095

an0n981 opened this issue Dec 24, 2014 · 30 comments
Labels

Comments

@an0n981
Copy link
Contributor

an0n981 commented Dec 24, 2014

The crash occured while scrolling the app list.

Log around the crash: https://gist.github.com/an0n981/e6cb5f1e370d57b64f17
If more logging is needed, I will try to reproduce the issue.

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

How are you, an0n981?

​Solving one out of memory results into a new out of memory.
Can you please check how much memory you have free?​

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

I am alright, how about yourself?

I had around 1.2 GB free at the time of the crash

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

I am fine, thanks!

I guess you didn't have this problem with 3.5.7 ?
There are not much changes, so you can easily downgrade if you like.

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

I don't know if it was caused by 3.5.8 or by TW rom, as I have just switched back to TW from AOSP. I previously had UI crashes with prevuozs XPrivacy versions. I can upload the logs if you would like.

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

Yes, please, but check first if the cause was an out of memory exception,
in that case that is all I need to know.

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

Logs sent via email, seem to not be related to out of memory error

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

All first crashes in the four log files have as
cause java.lang.OutOfMemoryError.

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

Might be a consequence of the changes for #2094

@M66B M66B added the bug label Dec 24, 2014
M66B added a commit that referenced this issue Dec 24, 2014
M66B added a commit that referenced this issue Dec 24, 2014
This will probaly have side effects

Refs #2095
@M66B M66B closed this as completed in b182527 Dec 24, 2014
@M66B
Copy link
Owner

M66B commented Dec 24, 2014

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

tmp_12858-screenshot_2014-12-24-21-12-05-1428324714
Now a lot of the icons don't show in the list view

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

Probably this: 52d70b0

I will later build a version without this change.

How is the speed?

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

Noticably faster. When scrolling and coming to a stop I used to have to wait a second for the list to finish processing, but now it's always ready as soon it stops scrolling.
Also worth noting, only 'Andy the Android' icons don't show up, and not all of them either. All apps with their own icon show up just fine.

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

Ah, interesting, 'Andy the Android' actually means no icon.
I was thinking about another problem, but I guess it is simpler.

M66B added a commit that referenced this issue Dec 24, 2014
@M66B
Copy link
Owner

M66B commented Dec 24, 2014

Please try again with this version:
https://crowd.xprivacy.eu/XPrivacy_3.5.8andy.apk

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

And thanks for testing on this Christmas evening!

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 24, 2014

Andy is back!
Testing XPrivacy is much more fun than watching 'Home Alone' for the 800th time, a weird Christmas tradition where I live.

@M66B
Copy link
Owner

M66B commented Dec 24, 2014

I feel the same about Home alone, etc ;-)
Just boring.

M66B added a commit that referenced this issue Dec 24, 2014
@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

I have noticed that there are now randomly icons missing while scrolling, when scrolling back through they reappear, there is no logic to which icons don't appear

M66B added a commit that referenced this issue Dec 25, 2014
@M66B
Copy link
Owner

M66B commented Dec 25, 2014

I guess this version will solve that:
https://crowd.xprivacy.eu/XPrivacy_3.5.8angry.apk

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

I liked the Andy version better thab the Angry version

@M66B
Copy link
Owner

M66B commented Dec 25, 2014

LOL
See, my commit ;-)

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

But now he is angry, we are back to him being missing, not as often as before, but still he does seem angry

@M66B
Copy link
Owner

M66B commented Dec 25, 2014

So, the problem is not fixed?

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

It is definitely geitting better, only 3 andys are missng. But no, not fixed completely

@M66B
Copy link
Owner

M66B commented Dec 25, 2014

Which applications?

@M66B
Copy link
Owner

M66B commented Dec 25, 2014

I have one angry Andy too.
But is the scrolling issue fixed?

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

Yes everything is working great, speed is amazing. The apps are com.samsung.indexservice, com.sec.android.app.SPenKeeper and com.sec.abdroid.providers.tasks.

M66B added a commit that referenced this issue Dec 25, 2014
@M66B
Copy link
Owner

M66B commented Dec 25, 2014

This version should even be faster and display all icons:
https://crowd.xprivacy.eu/XPrivacy_3.5.8happy.apk

@an0n981
Copy link
Contributor Author

an0n981 commented Dec 25, 2014

All the icons are happy again :)

@M66B
Copy link
Owner

M66B commented Dec 26, 2014

All changes are available in beta version 3.5.9
https://github.com/M66B/XPrivacy/releases/tag/3.5.9

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

No branches or pull requests

2 participants