Skip to content

🐛 fix: saturating audio + icon not loading#3

Merged
Nxssie merged 6 commits into
mainfrom
feature/test-introduction
Apr 25, 2026
Merged

🐛 fix: saturating audio + icon not loading#3
Nxssie merged 6 commits into
mainfrom
feature/test-introduction

Conversation

@Nxssie
Copy link
Copy Markdown
Owner

@Nxssie Nxssie commented Apr 25, 2026

Summary

  • Fix saturated/distorted audio: setPixelFormat was used for audio (it's a video option) — changed to setOption("sample_fmt", "s16")
  • Fix broken volume control: replaced unreliable hardware MASTER_GAIN with digital gain (broken on PulseAudio/PipeWire)
  • Fix sign-extension bug in digital volume gain loop that corrupted negative samples
  • Fix icon not loading: build script was looking for icon.png but asset is wren.png

Changes

  • FFmpegPlayer.kt: audio format fix, digital volume gain with proper byte masking, removed MASTER_GAIN
  • build-appimage.sh: icon file name corrected

Nxssie added 6 commits April 25, 2026 23:42
- setPixelFormat → setOption(sample_fmt, s16) for audio (was using video option)
- Replace broken MASTER_GAIN hardware control with digital gain (hardware is unreliable on PulseAudio/PipeWire)
- Fix sign-extension bug in digital volume gain loop (and 0xFF mask on high byte)
- Default digitalVolume to 1.0 instead of 0.5 (was halving signal)
- Fix icon lookup: use wren.png instead of icon.png in build script
- Add Msi and Dmg target formats to native distributions
- Add distro job to CI that builds on Linux, Windows, and macOS
- Upload artifacts for all three platforms
…recation warning

- checkout: v4 → v6
- setup-java: v4 → v5
- setup-gradle: v4 → v6
- upload-artifact: v4 → v6
@Nxssie Nxssie force-pushed the feature/test-introduction branch from 21ed949 to 6409e0f Compare April 25, 2026 22:42
@Nxssie Nxssie merged commit 9b1e8b8 into main Apr 25, 2026
5 checks passed
@Nxssie Nxssie deleted the feature/test-introduction branch April 25, 2026 22:50
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