Skip to content

Fix USB mic level gauge not showing on connect when mic source is PC (#2085)#2086

Merged
ten9876 merged 1 commit intomainfrom
aetherclaude/issue-2085
Apr 30, 2026
Merged

Fix USB mic level gauge not showing on connect when mic source is PC (#2085)#2086
ten9876 merged 1 commit intomainfrom
aetherclaude/issue-2085

Conversation

@aethersdr-agent
Copy link
Copy Markdown
Contributor

Summary

Fixes #2085

What was changed

Fix USB mic level gauge not showing on connect when mic source is PC (#2085)

Files modified

  • src/gui/PhoneCwApplet.cpp
 src/gui/PhoneCwApplet.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Generated by AetherClaude (automated agent for AetherSDR)

…2085)

The met_in_rx gate in updateMeters() was suppressing client-side PC mic
metering during RX. PC mic metering is driven by QAudioSource on the
client, independent of the radio's met_in_rx state, so skip the gate
when mic source is "PC".

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@aethersdr-agent aethersdr-agent Bot enabled auto-merge (squash) April 27, 2026 11:22
Copy link
Copy Markdown
Contributor Author

@aethersdr-agent aethersdr-agent Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Small, focused fix that correctly addresses the bug.

What the change does: When mic source is "PC", the client generates its own metering (since the radio always returns mic_level=0 for PC sources — see syncPhoneFromModel() at line 654). The old suppression logic blanked the gauge whenever met_in_rx was off and not transmitting, which inadvertently hid the PC mic level on connect. Adding the micSelection() != "PC" exception is the right fix — PC metering is client-authoritative and shouldn't be gated by the radio's met_in_rx flag.

Code quality: Uses QStringLiteral for the comparison, only touches the file in scope, no new risks introduced. The condition reads clearly with the && on its own line.

No issues found. Thanks for the fix, @aethersdr-agent!

@ten9876 ten9876 merged commit eecab89 into main Apr 30, 2026
5 checks passed
@ten9876 ten9876 deleted the aetherclaude/issue-2085 branch April 30, 2026 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0.9.1 USB Mic Not Initially Recognized

1 participant