mumble-voip/mumble

Iterate through positional audio program names only once

Aperta

#5712 aperta il 1 giu 2022

Vedi su GitHub
 (4 commenti) (2 reazioni) (0 assegnatari)C++ (1062 fork)batch import
clientfeature-requestgood first issue

Metriche repository

Star
 (5642 stelle)
Metriche merge PR
 (Merge medio 6g) (7 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga come i plugin audio posizionali di Mumble attualmente iterano sui PID e propone un refactoring in cui ogni plugin dichiara il nome del programma target, permettendo a Mumble di iterare la lista dei PID una sola volta e inviare al plugin corrispondente.
Tech stack
cpp
Dominio
Nessuno
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Mumble plugin architecture
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60