style: suppress ReturnCount on the processor's decision-table methods

dcalhoun · claude · dcalhoun · commit 70cf91132439 · 2026-07-24T09:13:35.000-04:00
processVideo and processImage are guard-clause decision tables where
early returns read clearer than nesting; suppress ReturnCount to match
the existing house style rather than fragment the logic.

Co-Authored-By: Claude Opus 4.8 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/posts/editor/GBKMediaUploadProcessor.kt b/WordPress/src/main/java/org/wordpress/android/ui/posts/editor/GBKMediaUploadProcessor.kt
@@ -77,6 +77,7 @@ class GBKMediaUploadProcessor(
         }
     }
 
+    @Suppress("ReturnCount")
     private suspend fun processVideo(file: File, filename: String): ProcessedProxyFile {
         if (mediaUtilsWrapper.isProhibitedVideoDuration(appContext, site, file)) {
             throw GBKMediaUploadException(
@@ -163,6 +164,7 @@ class GBKMediaUploadProcessor(
         composer.start()
     }
 
+    @Suppress("ReturnCount")
     private fun processImage(file: File, mimeType: String, filename: String): ProcessedProxyFile {
         // getOptimizedMedia returns null when optimization is disabled or a no-op. It can also
         // return the *input* path unchanged (GIF-like skips, decode failures inside

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ class GBKMediaUploadProcessor(`
`77`	`77`	`}`
`78`	`78`	`}`
`79`	`79`
	`80`	`+ @Suppress("ReturnCount")`
`80`	`81`	`private suspend fun processVideo(file: File, filename: String): ProcessedProxyFile {`
`81`	`82`	`if (mediaUtilsWrapper.isProhibitedVideoDuration(appContext, site, file)) {`
`82`	`83`	`throw GBKMediaUploadException(`
`@@ -163,6 +164,7 @@ class GBKMediaUploadProcessor(`
`163`	`164`	`composer.start()`
`164`	`165`	`}`
`165`	`166`
	`167`	`+ @Suppress("ReturnCount")`
`166`	`168`	`private fun processImage(file: File, mimeType: String, filename: String): ProcessedProxyFile {`
`167`	`169`	`// getOptimizedMedia returns null when optimization is disabled or a no-op. It can also`
`168`	`170`	`// return the input path unchanged (GIF-like skips, decode failures inside`