-
Notifications
You must be signed in to change notification settings - Fork 398
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
Show audio mixer identifier strings on the visualiser #4876
Conversation
Rationale is that it can no longer work without being in the audio hierarchy. A `TrackStore` should be used instead.
It doesn't support adjustments and is not in the expected location in the hierarchy. This only leads to confusion.
Remove broken `Handle` events These would potentially never be fired due to the fact the work that is going to fire them is run on the audio thread, before the binding may have completed.
Also pulls from `DrawableAudioMixer.Name`.
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.
Most changes that aren't dependencies look good, just one minor note.
@@ -18,7 +18,7 @@ public class DrawableAudioMixer : AudioContainer, IAudioMixer | |||
[BackgroundDependencyLoader] | |||
private void load(AudioManager audio) | |||
{ | |||
mixer = audio.CreateAudioMixer(); | |||
mixer = audio.CreateAudioMixer(Name); |
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.
Unfortunate that this is timing dependent (will only work if set prior to load). Not sure if it's worth spending the time to add support for arbitrary-timed changes. If we had initialisers then we could use that to make sure it's not an issue ever but alas...
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 sure it matters too much. Very rare you'd be setting the name of a drawable at any point after construction.
Found it hard to track what's what. This helps somewhat.
AudioMixer
visualiser #4875 to avoid merge conflicts.