jamulussoftware / jamulussoftware/jamulus

Investigate Android sound code issues reported by Codacy

Offen
#2,570 14 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
android bug
Vorherrschende Sprache
C
Sterne
1.1k
Forks
248
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
9

Beschreibung

**Describe the bug**

Codacy complains that the following variable is not used (and I don't see that this is false):

https://github.com/jamulussoftware/jamulus/blob/a9deb2259161b6cb86c12259f3e6f398189029ef/android/sound.cpp#L150

Also this warning:
![image](https://user-images.githubusercontent.com/20726856/161150084-72b0f33c-caeb-4a8b-a2b3-067e8888a37a.png)

I assume this is a false positive?
**To Reproduce**

See https://app.codacy.com/gh/ann0see/jamulus/file/68862695471/issues/source?bid=21822857&fileBranchId=21822857

**Expected behavior**

No Codacy warnings

**Screenshots**

See above

**Operating system**

Android

**Version of Jamulus**

Latest

**Additional context**

@j-santander @sthenos could you please have a look at these warnings too?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit der Untersuchung von android/sound.cpp um Zeile 150 und prüfe die beiden Warnungen im verknüpften Codacy-Bericht. Ermittle, ob die Variable tatsächlich ungenutzt ist und ob die zweite Warnung ein False Positive ist. Erledigt ist die Aufgabe, wenn die relevanten Warnungen behoben oder begründet sind und Codacy keine verbleibenden Warnungen für diesen Code meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
android, cpp
Bereich
audio-video-rtc, mobile-dev
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
50/100

Neue Issues direkt in Ihr Postfach

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