Skip to content

Conversation

@dimtpap
Copy link
Contributor

@dimtpap dimtpap commented Jan 15, 2026

Description

Fix the format specifiers in the message string for unsupported audio channels in the linux-pulseaudio plugin.

Motivation and Context

I was debugging an issue and the message in the log was not helpful.

How Has This Been Tested?

Add a PulseAudio output capture and target a sink with unsupported channels.
Before:

info: pulse-input: 	 channels not supported by OBS,using  instead for recording

After:

info: pulse-input: 9 channels not supported by OBS, using 2 instead for recording

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@RytoEX RytoEX added Bug Fix Non-breaking change which fixes an issue Linux Affects Linux Other OS (*nix) Other Unix-like systems that are not officially supported (e.g. OpenBSD) labels Jan 15, 2026
@dimtpap dimtpap force-pushed the pulse-log-formatting branch from 03c84f6 to 08d1516 Compare January 15, 2026 21:49
Copy link
Collaborator

@tytan652 tytan652 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RytoEX RytoEX added this to the OBS Studio 32.1 milestone Jan 16, 2026
@RytoEX RytoEX merged commit 9251ea5 into obsproject:master Jan 16, 2026
15 checks passed
@dimtpap dimtpap deleted the pulse-log-formatting branch January 16, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fix Non-breaking change which fixes an issue Linux Affects Linux Other OS (*nix) Other Unix-like systems that are not officially supported (e.g. OpenBSD)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants