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

cubeb: Change name to "Dolphin Emulator" #12063

Merged
merged 1 commit into from Jul 23, 2023

Conversation

SuperSamus
Copy link
Contributor

To avoid conflicts with KDE's file manager.

Before:
image

After:
image

To avoid conflicts with KDE's file manager.
@cobalt2727
Copy link

does that bug only happen if the audio backend is Cubeb?

@SuperSamus
Copy link
Contributor Author

Yes. It doesn't happen on Pulse or ALSA (checked on both versions).

@Naim2000
Copy link
Contributor

Just set it to pulse, looks alright
image

@JMC47
Copy link
Contributor

JMC47 commented Jul 22, 2023

Pfft, we were Dolphin first. They should change their name to Dolphin Folder Manager!!!!

/s

In all seriousness this seems fine at a glance.

@sepalani
Copy link
Contributor

I'm unfamiliar with this folder manager, what happen if you launch 2 Dolphin instances (i.e. a normal one and a portable one)? Are there two separate icons (since they're different apps) and if so, will this PR fix the bug?

@mbc07
Copy link
Contributor

mbc07 commented Jul 23, 2023

It's probably wise to add a comment in the code explaining why we're explicitly using full name to initialize Cubeb...

@SuperSamus
Copy link
Contributor Author

SuperSamus commented Jul 23, 2023

if you launch 2 Dolphin instances

The instances are stacked in the task manager.
image

Screenshot taken with 3 Dolphins open (only one playing a game).

EDIT: If you separate the icons, only the ones playing a game show the audio icon.

@SuperSamus
Copy link
Contributor Author

It's probably wise to add a comment in the code explaining why we're explicitly using full name to initialize Cubeb...

I don't think anyone looking at the code is going to wonder that?
And if they do, git blame exists.

@AdmiralCurtiss
Copy link
Contributor

Yeah I don't think that's necessary...

sepalani makes a good point though. Should we maybe append the process ID or something so the individual instances can be differentiated? What exactly does this ID string do, anyway?

@SuperSamus
Copy link
Contributor Author

What exactly does this ID string do, anyway?

It's the name shown when looking which applications are playing audio.

image

@cobalt2727
Copy link

In that case I would definitely agree that appending the process ID would be good, to make it easy to, say, turn the volume down on all but one of them.

(Maybe only append something if multiple instances are open? Is detecting that possible somehow?)

Copy link
Member

@JosJuice JosJuice left a comment

Choose a reason for hiding this comment

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

I think this change is fine as it is.

@SuperSamus
Copy link
Contributor Author

In that case I would definitely agree that appending the process ID would be good, to make it easy to, say, turn the volume down on all but one of them.

You already can.

image

@SuperSamus
Copy link
Contributor Author

SuperSamus commented Jul 23, 2023

Still, there is something weird going on KDE's side.

I tried having multiple instances of multiple applications, and playing audio on only one of them each.
In this image:
image

  • The first mpv is playing music, the second is not.
  • The first Dolphin Emulator instance is playing a game, and Dolphin file manager is "inheriting" the speaker icon (even though it's not even open).
  • However, the second Dolphin Emulator instance is not playing any games, and it doesn't have the speaker icon (as it should).

I feel like I should report it.

EDIT: https://bugs.kde.org/show_bug.cgi?id=472539

@AdmiralCurtiss
Copy link
Contributor

Well, if you figure out anything else let us know, for now let's just merge this as-is.

@AdmiralCurtiss AdmiralCurtiss merged commit dd3caa4 into dolphin-emu:master Jul 23, 2023
11 checks passed
@SuperSamus SuperSamus deleted the audio-name branch July 23, 2023 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
8 participants