Skip to content

Change in the range widget; and more fixes.#6

Open
andualem wants to merge 11 commits intojkpr:developfrom
andualem:master
Open

Change in the range widget; and more fixes.#6
andualem wants to merge 11 commits intojkpr:developfrom
andualem:master

Conversation

@andualem
Copy link

No description provided.

@jkpr
Copy link
Owner

jkpr commented Feb 26, 2018

Please describe the mUseLog null pointer exception. How to encounter/reproduce it? When did it start happening for you? Was it consistent for you? I have never seen it, and by contract it should never happen since onCreate is always called first and mUseLog is always initialized there.

@andualem
Copy link
Author

The null pointer exception occurs when the screen is rotated while the form is open. It also occurs when the screen resumes from sleep mode. In both of these cases, null pointer exception is thrown.

@jkpr
Copy link
Owner

jkpr commented Feb 27, 2018

I know you called it a temporary fix, but the solution is not to check for nullity and then call. The solution is to ensure that the mUseLog is initialized. A better attempt is to move it up higher in the onCreate method and to test that.

@jkpr
Copy link
Owner

jkpr commented Feb 27, 2018

It also occurs when the screen resumes from sleep mode.

This cannot be the entire story. There must be some other conditions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants