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

[AudioManager] Improve documentation #6352

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

Conversation

Jaechul-Lee
Copy link

Description of Change

Improve documentation

@github-actions github-actions bot added the API12 Platform : Tizen 9.0 / TFM: net6.0-tizen9.0 label Sep 25, 2024
/// Sets the sample format for the audio device.
/// This method allows developers to specify a desired <see cref="AudioSampleFormat"/> for
/// audio playback or recording. Setting an appropriate sample format is essential for
/// ensuring compatibility with the audio data being processed.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the last sentence is not that much helpful. could you remove that?

Copy link
Author

Choose a reason for hiding this comment

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

sure "Setting an appropriate sample format is essential for ensuring compatibility with the audio data being processed." will be removed

/// Sets the 'avoid resampling' property for the audio device.
/// This property controls whether the device should avoid resampling audio data during playback.
/// Enabling this feature can help preserve audio quality by preventing alterations to audio that
/// may happen during format conversions.
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess "during playback" is more proper instead of "during format conversions".

Copy link
Author

Choose a reason for hiding this comment

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

Okay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API12 Platform : Tizen 9.0 / TFM: net6.0-tizen9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants