Remove default params to flow extensions

Would rather be verbose and safe than introduce the default dispatcher unknowingly in situations where it would cause issues
cashapp · Sep 7, 2022 · 7aee4c6 · 7aee4c6
1 parent eae3c34
commit 7aee4c6
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/...outines-extensions/src/commonMain/kotlin/app/cash/sqldelight/coroutines/FlowExtensions.kt b/...outines-extensions/src/commonMain/kotlin/app/cash/sqldelight/coroutines/FlowExtensions.kt
@@ -58,7 +58,7 @@ fun <T : Any> Query<T>.asFlow(): Flow<Query<T>> = flow {
 
 @JvmOverloads
 fun <T : Any> Flow<Query<T>>.mapToOne(
-  context: CoroutineContext = Dispatchers.Default,
+  context: CoroutineContext,
 ): Flow<T> = map {
   withContext(context) {
     it.awaitAsOne()
@@ -68,7 +68,7 @@ fun <T : Any> Flow<Query<T>>.mapToOne(
 @JvmOverloads
 fun <T : Any> Flow<Query<T>>.mapToOneOrDefault(
   defaultValue: T,
-  context: CoroutineContext = Dispatchers.Default,
+  context: CoroutineContext,
 ): Flow<T> = map {
   withContext(context) {
     it.awaitAsOneOrNull() ?: defaultValue
@@ -77,7 +77,7 @@ fun <T : Any> Flow<Query<T>>.mapToOneOrDefault(
 
 @JvmOverloads
 fun <T : Any> Flow<Query<T>>.mapToOneOrNull(
-  context: CoroutineContext = Dispatchers.Default,
+  context: CoroutineContext,
 ): Flow<T?> = map {
   withContext(context) {
     it.awaitAsOneOrNull()
@@ -86,7 +86,7 @@ fun <T : Any> Flow<Query<T>>.mapToOneOrNull(
 
 @JvmOverloads
 fun <T : Any> Flow<Query<T>>.mapToOneNotNull(
-  context: CoroutineContext = Dispatchers.Default,
+  context: CoroutineContext,
 ): Flow<T> = mapNotNull {
   withContext(context) {
     it.awaitAsOneOrNull()
@@ -95,7 +95,7 @@ fun <T : Any> Flow<Query<T>>.mapToOneNotNull(
 
 @JvmOverloads
 fun <T : Any> Flow<Query<T>>.mapToList(
-  context: CoroutineContext = Dispatchers.Default,
+  context: CoroutineContext,
 ): Flow<List<T>> = map {
   withContext(context) {
     it.awaitAsList()