Skip to content
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

win-dshow: use the obs samplingRate #5033

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wkpark
Copy link

@wkpark wkpark commented Jul 26, 2021

dependency: obsproject/libdshowcapture#33

Description

set the desired sampling rate for the audio capture device using audio_output_get_sample_rate()

Motivation and Context

Please see obsproject/libdshowcapture#33

How Has This Been Tested?

obsproject/libdshowcapture#33

Types of changes

get the desired sampling rate using audio_output_get_sample_rate()

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.

@WizardCM WizardCM added the Enhancement Improvement to existing functionality label Jul 27, 2021
@WizardCM WizardCM changed the title WIP: use the obs samplingRate win-dshow: use the obs samplingRate Aug 21, 2021
@WizardCM WizardCM added the Feedback required Issue does not contain enough information or is missing the template label Sep 18, 2021
// get the obs sample_rate
uint32_t sampleRate = audio_output_get_sample_rate(obs_get_audio());

audioConfig.sampleRate =
Copy link
Member

Choose a reason for hiding this comment

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

Move the comment above the line to avoid clang-format having to break it up


audioConfig.sampleRate =
sampleRate; // try to set the desired samplerate
blog(LOG_INFO, "\tobs samplerate: %d", audioConfig.sampleRate);
Copy link
Member

Choose a reason for hiding this comment

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

Avoid using tabstops for log output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to existing functionality Feedback required Issue does not contain enough information or is missing the template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants