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

audio: Remove checkAudoBuffering #162

Closed

Conversation

prgmitchell
Copy link
Member

@prgmitchell prgmitchell commented Jun 4, 2024

Description

This removes the checkAudioBuffering function from the log analyzer.

Motivation and Context

In all of my time in support, apart from the application audio capture buffering issues which have been solved, this has rarely contributed to the interaction and almost always detracts from it. Nearly every instance of this error relates to sources that are not actually using audio or sources that hit max audio buffering, reset, and then had no issues all without the user noticing. Most see this critical error and assume it is the cause of their unrelated issue. This usually leads to a conversation explaining that if a user is not actually experiencing any audio issues you would expect (cutting out or audio going totally silent), this can simply be ignored anyways.

How Has This Been Tested?

Nothing to test on this one, just removing it.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)

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.

@prgmitchell prgmitchell force-pushed the removeAudioBufferCheck branch 3 times, most recently from 79aeb59 to ad2fef1 Compare June 9, 2024 01:52
Removes the audio buffering check
@prgmitchell prgmitchell force-pushed the removeAudioBufferCheck branch 2 times, most recently from fb67d49 to 0789f2c Compare June 9, 2024 02:01
@prgmitchell prgmitchell closed this Jun 9, 2024
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