Skip to content

fix: resolve compiler warnings across 6 files#469

Merged
datlechin merged 1 commit intomainfrom
fix/compiler-warnings
Mar 26, 2026
Merged

fix: resolve compiler warnings across 6 files#469
datlechin merged 1 commit intomainfrom
fix/compiler-warnings

Conversation

@datlechin
Copy link
Collaborator

Summary

  • Replace deprecated String(bytesNoCopy:length:encoding:freeWhenDone:) with UnsafeBufferPointer + String(decoding:as:) in SSH keyboard-interactive auth
  • Change var root to let root in RedisKeyTreeViewModel (reference is never reassigned)
  • Add explicit self. for sidebarViewModel in MainContentCoordinator+Navigation closure
  • Remove unused connectionName variable in MainContentView
  • Wrap main-actor-isolated calls in MainActor.assumeIsolated in DataGridView notification observers (already dispatched on queue: .main)
  • Remove redundant as? InMemoryRowProvider casts in DataGridView+RowActions (type already known)

Test plan

  • Build succeeds with zero warnings
  • SSH keyboard-interactive authentication still works
  • Redis key tree loads correctly
  • DataGrid row copy/paste operations unchanged

@datlechin datlechin merged commit 1716b38 into main Mar 26, 2026
2 checks passed
@datlechin datlechin deleted the fix/compiler-warnings branch March 26, 2026 13:05
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