mumble-voip/mumble

Iterate through positional audio program names only once

Ouverte

#5 712 ouverte le 1 juin 2022

Voir sur GitHub
 (4 commentaires) (2 réactions) (0 personne assignée)C++ (1 062 forks)batch import
clientfeature-requestgood first issue

Métriques du dépôt

Stars
 (5 642 étoiles)
Métriques de merge PR
 (Merge moyen 6j) (7 PRs mergées en 30 j)

Description

Context

Right now Mumble iterates through every program name of every PID for every positional audio plugin. This is quite wasted CPU time if either the PID count is high or plugin count is high, as the time complexity is in $\mathcal{O}(PIDNum \cdot pluginNum )$.

Example: https://github.com/mumble-voip/mumble/blob/dd5df45313a937359285327f556557159bab758e/plugins/gtav/gtav.cpp#L81

Description

A better solution would be if each plugin tells Mumble what program name it is looking for, so that only Mumble has to iterate over the PID-list once.

Mumble component

Client

OS-specific?

No

Additional information

This issue originated from https://github.com/mumble-voip/mumble/pull/5704#issuecomment-1141938952

Guide contributeur

Direction de recherche
Examinez comment les plugins audio positionnels de Mumble itèrent actuellement sur les PID et proposez un remaniement où chaque plugin déclare son nom de programme cible, permettant à Mumble d'itérer la liste des PID une seule fois et de distribuer au plugin correspondant.
Stack technique
cpp
Domaine
Aucun
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Mumble plugin architecture
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60