jamulussoftware / jamulussoftware/jamulus

windows/sound.cpp: Rename CheckDeviceCapabilities()

Offen
#1,409 9 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
good first issue refactoring
Vorherrschende Sprache
C
Sterne
1.1k
Forks
248
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
9

Beschreibung

**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

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie in windows/sound.cpp, indem Sie CheckDeviceCapabilities() lesen und nach dessen Aufrufstellen und Referenzen suchen. Wählen Sie einen Namen, der seine Aktualisierungen des Instanzzustands widerspiegelt, benennen Sie die Funktion durchgängig um und vergewissern Sie sich, dass keine Verweise auf den alten Namen verbleiben.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
audio-video-rtc
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.