Analysis: Use suppress in favor of suppress warnings for kotlin classes

The Kotlin stdlib related '@Suppress' annotation is preferable to the
Java lang related '@SuppressWarnings' annotation, if not for anything,
even just for consistency purposes. As such, the '@SuppressWarnings' is
being replaced with '@Suppress' on all Kotlin classes, wherever it is
found.

Author: ParaskP7

WordPress/src/main/java/org/wordpress/android/AppInitializer.kt

@@ -533,7 +533,7 @@ class AppInitializer @Inject constructor(
         }
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe(threadMode = ThreadMode.MAIN)
     fun onAuthenticationChanged(event: OnAuthenticationChanged) {
         if (accountStore.hasAccessToken()) {

WordPress/src/main/java/org/wordpress/android/WordPress.kt

@@ -58,15 +58,15 @@ abstract class WordPress : MultiDexApplication() {
         @JvmStatic
         fun getRestClientUtils() = AppInitializer.restClientUtils
 
-        @SuppressWarnings("FunctionNaming")
+        @Suppress("FunctionNaming")
         @JvmStatic
         fun getRestClientUtilsV1_1() = AppInitializer.restClientUtilsV1_1
 
-        @SuppressWarnings("FunctionNaming")
+        @Suppress("FunctionNaming")
         @JvmStatic
         fun getRestClientUtilsV1_2() = AppInitializer.restClientUtilsV1_2
 
-        @SuppressWarnings("FunctionNaming")
+        @Suppress("FunctionNaming")
         @JvmStatic
         fun getRestClientUtilsV1_3() = AppInitializer.restClientUtilsV1_3
 

WordPress/src/main/java/org/wordpress/android/ui/JetpackRemoteInstallViewModel.kt

@@ -147,8 +147,8 @@ class JetpackRemoteInstallViewModel
     }
 
     // Network Callbacks
+    @Suppress("unused")
     @Subscribe(threadMode = ThreadMode.BACKGROUND)
-    @SuppressWarnings("unused")
     fun onEventsUpdated(event: OnJetpackInstalled) {
         val site = siteModel ?: return
         if (event.isError) {

WordPress/src/main/java/org/wordpress/android/ui/engagement/GetLikesUseCase.kt

@@ -46,10 +46,8 @@ import kotlin.coroutines.suspendCoroutine
 class GetLikesUseCase @Inject constructor(
     private val networkUtilsWrapper: NetworkUtilsWrapper,
     private val dispatcher: Dispatcher,
-    @SuppressWarnings("Unused")
-    val commentStore: CommentStore,
-    @SuppressWarnings("Unused")
-    val postStore: PostStore,
+    @Suppress("Unused") val commentStore: CommentStore,
+    @Suppress("Unused") val postStore: PostStore,
     val accountStore: AccountStore
 ) {
     private var getLikesContinuations = mutableMapOf<String, Continuation<OnChanged<*>>>()
@@ -76,7 +74,7 @@ class GetLikesUseCase @Inject constructor(
         getLikes(COMMENT_LIKE, this, fingerPrint, paginationParams)
     }
 
-    @SuppressWarnings("ComplexMethod", "NestedBlockDepth", "LoopWithTooManyJumpStatements")
+    @Suppress("ComplexMethod", "NestedBlockDepth", "LoopWithTooManyJumpStatements")
     private suspend fun getLikes(
         category: LikeCategory,
         flow: FlowCollector<GetLikesState>,

WordPress/src/main/java/org/wordpress/android/ui/jetpack/JetpackCapabilitiesUseCase.kt

@@ -96,8 +96,8 @@ class JetpackCapabilitiesUseCase @Inject constructor(
         )
     }
 
+    @Suppress("unused")
     @Subscribe(threadMode = ThreadMode.MAIN)
-    @SuppressWarnings("unused")
     fun onJetpackCapabilitiesFetched(event: OnJetpackCapabilitiesFetched) {
         continuation[event.remoteSiteId]?.let {
             continuation.remove(event.remoteSiteId)

WordPress/src/main/java/org/wordpress/android/ui/main/MeFragment.kt

@@ -586,7 +586,7 @@ class MeFragment : Fragment(R.layout.me_fragment), OnScrollToTopListener {
         }
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe(threadMode = ThreadMode.MAIN)
     fun onAccountChanged(event: OnAccountChanged?) {
         binding?.refreshAccountDetails()

WordPress/src/main/java/org/wordpress/android/ui/notifications/NotificationsListFragmentPage.kt

@@ -443,7 +443,7 @@ class NotificationsListFragmentPage : ViewPagerFragment(R.layout.notifications_l
         notesAdapter!!.addAll(event.notes, true)
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe(threadMode = MAIN)
     fun onEventMainThread(error: NotificationsRefreshError?) {
         if (isAdded) {

WordPress/src/main/java/org/wordpress/android/ui/plans/PlansViewModel.kt

@@ -89,8 +89,8 @@ class PlansViewModel @Inject constructor(
         _plans.value = _cachedPlans.value
     }
 
+    @Suppress("unused")
     @Subscribe(threadMode = ThreadMode.MAIN)
-    @SuppressWarnings("unused")
     fun onPlanOffersFetched(event: OnPlanOffersFetched) {
         if (event.isError && event.planOffers?.isEmpty() == false) {
             _listStatus.value = PlansListStatus.ERROR_WITH_CACHE

WordPress/src/main/java/org/wordpress/android/ui/posts/PrepublishingCategoriesFragment.kt

@@ -205,13 +205,13 @@ class PrepublishingCategoriesFragment : Fragment(R.layout.prepublishing_categori
         }
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe(threadMode = MAIN)
     fun onTermUploaded(event: OnTermUploaded) {
         viewModel.onTermUploadedComplete(event)
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe(threadMode = MAIN)
     fun onTaxonomyChanged(event: OnTaxonomyChanged) {
         viewModel.onTaxonomyChanged(event)

WordPress/src/main/java/org/wordpress/android/ui/posts/editor/StorePostViewModel.kt

@@ -187,13 +187,13 @@ class StorePostViewModel
         _onFinish.postValue(Event(state))
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe
     fun onPostUploaded(event: OnPostUploaded) {
         hideSavingProgressDialog()
     }
 
-    @SuppressWarnings("unused")
+    @Suppress("unused")
     @Subscribe
     fun onPostChanged(event: OnPostChanged) {
         hideSavingProgressDialog()