jamulussoftware / jamulussoftware/jamulus

windows/sound.cpp: Rename CheckDeviceCapabilities()

Aperta
#1,409 9 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

good first issue refactoring
Lingua principale
C
Stelle
1.1k
Fork
248
Merge medio
2g 3h
PR unite (30g)
9

Descrizione

**Describe the bug**
`CheckDeviceCapabilities()` in `windows/sound.cpp` does more than what the name suggests: It has side effects such as updating instance variables such as `lNumInChan`.

**Expected behavior**

Have a name which makes it clear that updates happen.

**Version of Jamulus**
d5d4691221b76bc648d605dac867aeaa561d0a52

cc @ann0see @pljones

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia in windows/sound.cpp leggendo CheckDeviceCapabilities() e cercando i relativi punti di chiamata e riferimenti. Scegli un nome che rifletta i suoi aggiornamenti allo stato dell’istanza, rinomina la funzione in modo coerente e verifica che non rimangano riferimenti al vecchio nome.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
audio-video-rtc
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.