Skip to content

Commit

Permalink
Remove some almost-never-used and therefore untested code, and remove…
Browse files Browse the repository at this point in the history
… a potentially misleading comment from the ALSA backend.
  • Loading branch information
mikebrady committed Apr 24, 2024
1 parent 5d24d84 commit c0f5112
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions audio_alsa.c
Original file line number Diff line number Diff line change
Expand Up @@ -938,9 +938,9 @@ static int prepare_mixer() {
} else {
// use the linear scale and do the db conversion ourselves
warn("The hardware mixer specified -- \"%s\" -- does not have "
"a dB volume scale.",
"a dB volume scale, and so can not be used by Shairport Sync.",
alsa_mix_ctrl);

/*
if ((response = snd_ctl_open(&ctl, alsa_mix_dev, 0)) < 0) {
warn("Cannot open control \"%s\"", alsa_mix_dev);
}
Expand All @@ -961,9 +961,10 @@ static int prepare_mixer() {
// we know it can do dB stuff
audio_alsa.parameters = &parameters; // likewise the parameters stuff
} else {
debug(1, "Cannot get the dB range from the volume control \"%s\"", alsa_mix_ctrl);
debug(1, "Cannot get a dB range from the volume control \"%s\"", alsa_mix_ctrl);
}
}
*/
}
}
if (((config.alsa_use_hardware_mute == 1) &&
Expand Down

0 comments on commit c0f5112

Please sign in to comment.