Add `synchronized` keyword to AppDatabase #16

Manually merged
vanitasvitae merged 1 commits from adiaholic/Mercury-IM:sync into master 2 years ago
Collaborator

Prevent from accidentally creating two instances of the same database as it is possible in a multi-threaded environment.

Prevent from accidentally creating two instances of the same database as it is possible in a multi-threaded environment.
Owner

I'm not sure if this is really necessary, as dagger (which is the only party accessing the database constructor) should be thread safe.

It might still be a good idea in general, so I'll happily merge it nontheless 👍

I'm not sure if this is really necessary, as dagger (which is the only party accessing the database constructor) should be thread safe. It might still be a good idea in general, so I'll happily merge it nontheless :+1:
vanitasvitae closed this pull request 2 years ago
adiaholic deleted branch sync 2 years ago
The pull request has been manually merged as c68a0b992d.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.