Skip to content

Commit 0ddcf0b

Browse files
committed
Remove unnecessary dispatcher switch in DefaultRecentSearchRepository
As mentioned in the following discussion: #818 Room already takes care of switching dispatchers if the DAO method is correctly annotated with the `suspend` modifier.
1 parent b989d3a commit 0ddcf0b

File tree

1 file changed

+6
-13
lines changed

1 file changed

+6
-13
lines changed

core/data/src/main/java/com/google/samples/apps/nowinandroid/core/data/repository/DefaultRecentSearchRepository.kt

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,28 +20,21 @@ import com.google.samples.apps.nowinandroid.core.data.model.RecentSearchQuery
2020
import com.google.samples.apps.nowinandroid.core.data.model.asExternalModel
2121
import com.google.samples.apps.nowinandroid.core.database.dao.RecentSearchQueryDao
2222
import com.google.samples.apps.nowinandroid.core.database.model.RecentSearchQueryEntity
23-
import com.google.samples.apps.nowinandroid.core.network.Dispatcher
24-
import com.google.samples.apps.nowinandroid.core.network.NiaDispatchers.IO
25-
import kotlinx.coroutines.CoroutineDispatcher
2623
import kotlinx.coroutines.flow.Flow
2724
import kotlinx.coroutines.flow.map
28-
import kotlinx.coroutines.withContext
2925
import kotlinx.datetime.Clock
3026
import javax.inject.Inject
3127

3228
class DefaultRecentSearchRepository @Inject constructor(
3329
private val recentSearchQueryDao: RecentSearchQueryDao,
34-
@Dispatcher(IO) private val ioDispatcher: CoroutineDispatcher,
3530
) : RecentSearchRepository {
3631
override suspend fun insertOrReplaceRecentSearch(searchQuery: String) {
37-
withContext(ioDispatcher) {
38-
recentSearchQueryDao.insertOrReplaceRecentSearchQuery(
39-
RecentSearchQueryEntity(
40-
query = searchQuery,
41-
queriedDate = Clock.System.now(),
42-
),
43-
)
44-
}
32+
recentSearchQueryDao.insertOrReplaceRecentSearchQuery(
33+
RecentSearchQueryEntity(
34+
query = searchQuery,
35+
queriedDate = Clock.System.now(),
36+
),
37+
)
4538
}
4639

4740
override fun getRecentSearchQueries(limit: Int): Flow<List<RecentSearchQuery>> =

0 commit comments

Comments
 (0)