-
Notifications
You must be signed in to change notification settings - Fork 6
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
Impl Deref for newtypes #71
base: main
Are you sure you want to change the base?
Conversation
The key inputs in the audio control and parameter adjustment functions have been updated. They've been changed to use ButtonInput instead of Input, and the specific KeyCode names have been adjusted to the recently updated KeyCode identifications.
fn stop(&self) { | ||
self.event_instance.stop(StopMode::AllowFadeout).unwrap(); | ||
self.0.stop(StopMode::AllowFadeout).unwrap(); | ||
} |
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.
Here is an example of what I was talking about in the PR description
Can you elaborate what your initial goal was with this PR? |
Mainly to reduce the amount of code the user needs to write. This is easiest to see in the diff of the parameters example |
Wanted to get opinions on this:
On
FmodStudio
it's a clear improvement in my eyes.But on
AudioSource
it gets a bit hairy (internally, user-facing looks ok) becauseAudioSinkPlayback
contains some function members with the same name as the ones inEventInstance
. So you still need the.0
syntax in some places (internally).