Temporary remove audio switching function

[joshuatam]

Temporary remove this feature, as it behaves differently across games.

Examples:
Subnautica: Working
Nidhogg: Not working

Likely games without proper audio switching handling would fail.

This feature will be continue to work on #376

Summary by CodeRabbit

• Changes
  • Removed automatic audio component restart when audio devices are added or removed. Audio device changes will no longer trigger system-level audio restarts.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Audio device callback handling and automatic audio component restart logic are removed from XEnvironment. Fields for managing AudioDeviceCallback, audio manager state, and the restartAudioComponent helper method are deleted, eliminating the automatic restart behavior triggered by audio device addition or removal events.

Changes

Cohort / File(s)	Change Summary
Audio Device Callback Removal app/src/main/java/com/winlator/xenvironment/XEnvironment.java	Removed audio manager field, AudioDeviceCallback implementation, and device callback registration from constructor. Deleted restartAudioComponent() helper method and all related calls. Audio device change events no longer trigger component restarts.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• fixed audio stopping after pause app #375: Related audio device callback modification—that PR removed pause/resume toggling while this PR removes the entire callback registration and restart mechanism from XEnvironment.

Poem

🐰 Hop, hop—audio threads now rest,
No callbacks jingle when devices change their nest.
Simpler paths for sounds to play,
Less complexity leads the way! 🎵

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Temporary remove audio switching function' accurately describes the main change - removing audio device callback logic and audio management. The description aligns with the code changes that eliminate dynamic audio device switching.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ea26792 and bb7d330.

📒 Files selected for processing (1)

• app/src/main/java/com/winlator/xenvironment/XEnvironment.java

💤 Files with no reviewable changes (1)

• app/src/main/java/com/winlator/xenvironment/XEnvironment.java

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}