Fix ktlint (#311)

Author: siarhei-luskanau

build.gradle.kts

@@ -49,7 +49,12 @@ allprojects {
     }
   }
 
-  apply(plugin = rootProject.libs.plugins.spotless.get().pluginId)
+  apply(
+    plugin =
+      rootProject.libs.plugins.spotless
+        .get()
+        .pluginId,
+  )
   configure<SpotlessExtension> {
     kotlin {
       target("**/*.kt")
@@ -59,8 +64,7 @@ allprojects {
           listOf(
             libs.ktlintComposeRules.get().toString(),
           ),
-        )
-        .editorConfigOverride(
+        ).editorConfigOverride(
           mapOf(
             // Disabled because paging-* filenames should be identical to that of AndroidX Paging.
             "ktlint_standard_filename" to "disabled",

paging-common/src/commonMain/kotlin/app/cash/paging/CombinedLoadStates.kt

@@ -37,12 +37,10 @@ fun createCombinedLoadStates(
   append: LoadState,
   source: LoadStates,
   mediator: LoadStates? = null,
-): CombinedLoadStates {
-  return CombinedLoadStates(
-    refresh,
-    prepend,
-    append,
-    source,
-    mediator,
-  )
-}
+): CombinedLoadStates = CombinedLoadStates(
+  refresh,
+  prepend,
+  append,
+  source,
+  mediator,
+)

paging-common/src/commonMain/kotlin/app/cash/paging/LoadStates.kt

@@ -37,10 +37,8 @@ fun LoadStates.copy(
   refresh: LoadState = this.refresh,
   prepend: LoadState = this.prepend,
   append: LoadState = this.append,
-): LoadStates {
-  return LoadStates(
-    refresh,
-    prepend,
-    append,
-  )
-}
+): LoadStates = LoadStates(
+  refresh,
+  prepend,
+  append,
+)

paging-common/src/commonMain/kotlin/app/cash/paging/Pager.kt

@@ -42,23 +42,19 @@ fun <Key : Any, Value : Any> createPager(
   initialKey: Key? = null,
   remoteMediator: RemoteMediator<Key, Value>?,
   pagingSourceFactory: () -> PagingSource<Key, Value>,
-): Pager<Key, Value> {
-  return Pager(
-    config,
-    initialKey,
-    remoteMediator,
-    pagingSourceFactory,
-  )
-}
+): Pager<Key, Value> = Pager(
+  config,
+  initialKey,
+  remoteMediator,
+  pagingSourceFactory,
+)
 
 fun <Key : Any, Value : Any> createPager(
   config: PagingConfig,
   initialKey: Key? = null,
   pagingSourceFactory: () -> PagingSource<Key, Value>,
-): Pager<Key, Value> {
-  return Pager(
-    config,
-    initialKey,
-    pagingSourceFactory,
-  )
-}
+): Pager<Key, Value> = Pager(
+  config,
+  initialKey,
+  pagingSourceFactory,
+)

paging-common/src/commonMain/kotlin/app/cash/paging/PagingConfig.kt

@@ -60,16 +60,14 @@ fun createPagingConfig(
   initialLoadSize: Int = pageSize * 3,
   maxSize: Int = MAX_SIZE_UNBOUNDED,
   jumpThreshold: Int = COUNT_UNDEFINED,
-): PagingConfig {
-  return PagingConfig(
-    pageSize,
-    prefetchDistance,
-    enablePlaceholders,
-    initialLoadSize,
-    maxSize,
-    jumpThreshold,
-  )
-}
+): PagingConfig = PagingConfig(
+  pageSize,
+  prefetchDistance,
+  enablePlaceholders,
+  initialLoadSize,
+  maxSize,
+  jumpThreshold,
+)
 
 @Suppress("MinMaxConstant")
 const val MAX_SIZE_UNBOUNDED: Int = Int.MAX_VALUE

paging-common/src/commonMain/kotlin/app/cash/paging/PagingSource.kt

@@ -86,9 +86,7 @@ expect class PagingSourceLoadResultError<Key : Any, Value : Any>(
 
 fun <Key : Any, Value : Any> PagingSourceLoadResultError<Key, Value>.copy(
   throwable: Throwable = this.throwable,
-): PagingSourceLoadResultError<Key, Value> {
-  return PagingSourceLoadResultError(throwable)
-}
+): PagingSourceLoadResultError<Key, Value> = PagingSourceLoadResultError(throwable)
 
 expect class PagingSourceLoadResultInvalid<Key : Any, Value : Any>()
 
@@ -128,30 +126,26 @@ fun <Key : Any, Value : Any> createPagingSourceLoadResultPage(
   nextKey: Key?,
   itemsBefore: Int = COUNT_UNDEFINED,
   itemsAfter: Int = COUNT_UNDEFINED,
-): PagingSourceLoadResultPage<Key, Value> {
-  return PagingSourceLoadResultPage(
-    data,
-    prevKey,
-    nextKey,
-    itemsBefore,
-    itemsAfter,
-  )
-}
+): PagingSourceLoadResultPage<Key, Value> = PagingSourceLoadResultPage(
+  data,
+  prevKey,
+  nextKey,
+  itemsBefore,
+  itemsAfter,
+)
 
 fun <Key : Any, Value : Any> PagingSourceLoadResultPage<Key, Value>.copy(
   data: List<Value> = this.data,
   prevKey: Key? = this.prevKey,
   nextKey: Key? = this.nextKey,
   itemsBefore: Int = this.itemsBefore,
   itemsAfter: Int = this.itemsAfter,
-): PagingSourceLoadResultPage<Key, Value> {
-  return PagingSourceLoadResultPage(
-    data,
-    prevKey,
-    nextKey,
-    itemsBefore,
-    itemsAfter,
-  )
-}
+): PagingSourceLoadResultPage<Key, Value> = PagingSourceLoadResultPage(
+  data,
+  prevKey,
+  nextKey,
+  itemsBefore,
+  itemsAfter,
+)
 
 const val COUNT_UNDEFINED: Int = Int.MIN_VALUE

paging-runtime-uikit/src/iosMain/kotlin/app/cash/paging/PagingCollectionViewController.kt

@@ -23,13 +23,9 @@ class PagingCollectionViewController<T : Any> {
   private var collectionView: UICollectionView? = null
 
   private val diffCallback = object : DiffUtil.ItemCallback<T>() {
-    override fun areItemsTheSame(oldItem: T, newItem: T): Boolean {
-      return oldItem == newItem
-    }
+    override fun areItemsTheSame(oldItem: T, newItem: T): Boolean = oldItem == newItem
 
-    override fun areContentsTheSame(oldItem: T, newItem: T): Boolean {
-      return oldItem == newItem
-    }
+    override fun areContentsTheSame(oldItem: T, newItem: T): Boolean = oldItem == newItem
   }
 
   private val differ = AsyncPagingDataDiffer(

samples/repo-search/android-composeui/build.gradle.kts

@@ -30,7 +30,10 @@ android {
   }
 
   composeOptions {
-    kotlinCompilerExtensionVersion = libs.androidx.compose.compiler.get().version
+    kotlinCompilerExtensionVersion =
+      libs.androidx.compose.compiler
+        .get()
+        .version
   }
 }
 

samples/repo-search/shared/src/commonMain/kotlin/app/cash/paging/samples/reposearch/RepoSearchPresenter.kt

@@ -44,16 +44,14 @@ class RepoSearchPresenter {
     }
   }
 
-  fun produceViewModels(events: Flow<Event>): Flow<ViewModel> {
-    return events.map { event ->
-      when (event) {
-        is Event.SearchTerm -> {
-          latestSearchTerm = event.searchTerm
-          if (event.searchTerm.isEmpty()) {
-            ViewModel.Empty
-          } else {
-            ViewModel.SearchResults(latestSearchTerm, pager.flow)
-          }
+  fun produceViewModels(events: Flow<Event>): Flow<ViewModel> = events.map { event ->
+    when (event) {
+      is Event.SearchTerm -> {
+        latestSearchTerm = event.searchTerm
+        if (event.searchTerm.isEmpty()) {
+          ViewModel.Empty
+        } else {
+          ViewModel.SearchResults(latestSearchTerm, pager.flow)
         }
       }
     }

samples/repo-search/shared/src/iosMain/kotlin/app/cash/paging/samples/reposearch/exposed.kt

@@ -7,9 +7,7 @@ import kotlinx.coroutines.flow.MutableSharedFlow
 fun exposedTypes(
   pagingCollectionViewController: PagingCollectionViewController<*>,
   mutableSharedFlow: MutableSharedFlow<*>,
-) {
-  throw AssertionError()
-}
+): Unit = throw AssertionError()
 
 @Suppress("unused") // Used to export types to Objective-C / Swift.
 fun <T> mutableSharedFlow(extraBufferCapacity: Int) = MutableSharedFlow<T>(extraBufferCapacity = extraBufferCapacity)