Skip to content

Fix compiler warnings in NetworkingTools and OverlayDisplaySettingsView - #307

Closed
hossain-khan wants to merge 1 commit into
mainfrom
fix/compiler-and-adaptive-warnings
Closed

Fix compiler warnings in NetworkingTools and OverlayDisplaySettingsView#307
hossain-khan wants to merge 1 commit into
mainfrom
fix/compiler-and-adaptive-warnings

Conversation

@hossain-khan

@hossain-khan hossain-khan commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR addresses two compilation warnings:

1. INVISIBLE_REFERENCE compiler warning in NetworkingTools.kt

  • Issue: Slack's EitherNet marks HttpFailure.tags with internal visibility. Accessing it via @Suppress("INVISIBLE_MEMBER", "INVISIBLE_REFERENCE") prompted a Kotlin 2.x compiler warning about unspecified compiler behavior.
  • Fix: Replaced compiler error suppression with safe runtime reflection to access tags, eliminating the warning cleanly.

2. currentWindowAdaptiveInfo deprecation warning in OverlayDisplaySettingsView.kt

  • Issue: currentWindowAdaptiveInfo() was deprecated in androidx.compose.material3.adaptive:1.3.0 in favor of supporting Large and Extra-Large breakpoints.
  • Fix: Passed supportLargeAndXLargeWidth = true and scoped deprecation suppression on the composable default argument.

Verification

  • Formatted via ./gradlew formatKotlin
  • Verified with ./gradlew lintKotlin testDebugUnitTest (all tests pass)
  • Verified build with ./gradlew compileDebugKotlin compileReleaseKotlin (both debug and release compile cleanly with zero warnings)

@hossain-khan

Copy link
Copy Markdown
Collaborator Author

Hmm! can't take it now, requires bit more through testing.

@hossain-khan
hossain-khan deleted the fix/compiler-and-adaptive-warnings branch August 15, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant