Skip to content

CI run: audio-only Windows fix (mirror of upstream #255) - #1

Closed
muthuraj57 wants to merge 1 commit into
masterfrom
audio-only-windows
Closed

muthuraj57 wants to merge 1 commit into
masterfrom
audio-only-windows

Conversation

@muthuraj57

Copy link
Copy Markdown
Owner

Runs the upstream Build and Test workflow (natives for Windows x64/arm64, Linux, macOS; JVM tests; lint) on the fork, since the upstream PR's CI waits for maintainer approval. Not for merging yet; upstream PR: kdroidFilter#255.

…ontent changes

- OpenMedia no longer fails with MF_E_INVALIDSTREAMNUMBER when the source has
  no video stream: the video stream is optional, and media without both video
  and audio is still rejected.
- The audio thread now sees MF_SOURCE_READERF_ENDOFSTREAM (the flag was checked
  after the null-sample early return) and, for audio-only media, sets EOF.
- Seeking after the audio thread exited (end of media) restarts it.
- The producer keeps running after onPlaybackEnded, so seek + play after the
  end works and the next end is reported again.
- Audio-only position/progress come from GetMediaPosition.
- NativeLibraryLoader compared only the file size, so an updated DLL of the same
  size kept the stale cached copy; it now compares the content.

Tested on Windows 11 (x64) with MP3, WAV and Ogg Opus files (duration,
position, seek, end, replay) and an HLS video (unchanged).
@muthuraj57

Copy link
Copy Markdown
Owner Author

Superseded by the tc-main CI PR.

@muthuraj57 muthuraj57 closed this Oct 24, 2026
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.

2 participants