Skip to content

Replace Context with ContextWrapper#3692

Draft
cmelchior wants to merge 4 commits intomainfrom
cm/context-wrapper
Draft

Replace Context with ContextWrapper#3692
cmelchior wants to merge 4 commits intomainfrom
cm/context-wrapper

Conversation

@cmelchior
Copy link
Copy Markdown
Contributor

Fixes #3530

This PR replaces our stored Context with a ContextWrapper. This means we use the parsed Context when calling init to retrieve the needed information and then store that.

It has the following benefits:

  • Should make Realm works better with Instant Run (Cannot find solid proof of this, but there was a warning in IntelliJ about it)
  • Will remove the Android Context from RealmConfiguration/SyncConfiguration/BaseRealm making it easier to support JVM in the future.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Avoid storing reference to ApplicationContext

2 participants