-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 Docs #2457
Audio Docs #2457
Conversation
/// <summary> | ||
/// Returns the name of this AudioCategoty | ||
/// </summary> | ||
/// <returns>Friendly name of the AudioCategoty</returns> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Categoty happens twice here
/// <param name="emitter">The emitter to calculate.</param> | ||
/// <remarks> | ||
/// <para>This must be called before Play().</para> | ||
/// <para>Calling this method automatically converts the sound to monoaural and sets the speaker mix for any sound played by this cue to a value calculated with the listener's and emitter's positions. Any stereo information in the sound will be discarded.</para> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not as wide, but still pretty wide
@@ -116,16 +140,22 @@ public void Play() | |||
PlatformPlay(); | |||
} | |||
|
|||
/// <summary>Resumes playback for a SoundEffectInstance.</summary> | |||
/// <remarks>Only has effect on an SoundEffectInstance in a paused state.</remarks> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"on an", Maybe instead "Only has an effect on a ..."
English review complete :) |
@tomspilman - Finished making the changes that @danzel suggested. Thanks again, Dave! |
First run at some documentation for classes in the Microsoft.Xna.Framework.Audio namespace.
I also took care of replacing the old licenses while I was at it.
A contribution to #2378