Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added null check, fix #560 #561

Closed
wants to merge 2 commits into from
Closed

Added null check, fix #560 #561

wants to merge 2 commits into from

Conversation

ale5000-git
Copy link
Member

@ale5000-git ale5000-git commented Jun 20, 2018 •
edited

See title.

@ale5000-git ale5000-git mentioned this pull request Jul 23, 2018
Closed
@mar-v-in
Copy link
Member

Should be fixed through 7bbdcac ?

@ale5000-git
Copy link
Member Author

ale5000-git commented Aug 28, 2018 •
edited

@mar-v-in: The problem has been fixed.

PS: I have just a note, I'm not sure if it is possible but in case the return value of cm.getActiveNetworkInfo() change between line 371 and 372 doesn't it remain in a inconsistent state with the current code?
For example if it is null on line 371 and not null on line 372.

Here.

@ale5000-git ale5000-git deleted the patch-1 branch August 28, 2018 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ale5000-git @mar-v-in