Skip to content

[pull] master from MusicPlayerDaemon:master #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 23 commits into from
Jun 30, 2025

Conversation

pull[bot]
Copy link

@pull pull bot commented Jun 30, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.2)

Can you help keep this open source service alive? 💖 Please sponsor : )

Summary by Sourcery

Update read_tags tool to use a command-line parser with config file and verbose logging support, overhaul network resolver error handling to use std::system_error, enhance decoder plugins, and apply various code cleanups and metadata updates.

New Features:

  • read_tags tool now accepts --config and --verbose flags and initializes plugins and logging via a command-line parser
  • Network Resolver now throws std::system_error with a custom error_category on getaddrinfo failures

Enhancements:

  • Add full_scan option and enforce seekable mode in Mpg123 decoder plugin
  • Recognize ".aif" extension for AIFF format in the Ffmpeg decoder
  • Introduce pure string_view overload for StringAfterPrefix
  • Streamline PlaylistVector lookup with string_view and simplify its mark logic
  • Fix format string in Configured database error message
  • Include cmdline dependency in test build configuration

Chores:

  • Mass-update author email metadata across source and test files

MaxKellermann and others added 23 commits June 10, 2025 15:54
Add a "*.aif" as a special case of the "aiff" demuxer.

Closes #2313
This has been implemented in commit b9488d0
find() takes a std::string_view, and passing a C string will
implicitly add an unnecessary strlen() call.
The "mark" field was not updated when scanning a playlist file that
had not been modified, causing playlists to disappear randomly.

Closes #2322
Use designated initializers instead.
@pull pull bot locked and limited conversation to collaborators Jun 30, 2025
@pull pull bot added the ⤵️ pull label Jun 30, 2025
@pull pull bot merged commit 677b75c into CartoonFan:master Jun 30, 2025
1 check was pending
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants