mirror of
https://github.com/overte-org/overte.git
synced 2025-08-11 06:22:10 +02:00
Set AEC enabled by default
This commit is contained in:
parent
3c79a79487
commit
5a8a5bbc55
4 changed files with 17 additions and 11 deletions
|
@ -3,8 +3,8 @@ package io.highfidelity.hifiinterface.fragment;
|
||||||
import android.content.SharedPreferences;
|
import android.content.SharedPreferences;
|
||||||
import android.media.audiofx.AcousticEchoCanceler;
|
import android.media.audiofx.AcousticEchoCanceler;
|
||||||
import android.os.Bundle;
|
import android.os.Bundle;
|
||||||
import android.preference.Preference;
|
|
||||||
import android.preference.PreferenceFragment;
|
import android.preference.PreferenceFragment;
|
||||||
|
import android.preference.PreferenceManager;
|
||||||
import android.support.annotation.Nullable;
|
import android.support.annotation.Nullable;
|
||||||
|
|
||||||
import io.highfidelity.hifiinterface.R;
|
import io.highfidelity.hifiinterface.R;
|
||||||
|
@ -18,17 +18,23 @@ public class SettingsFragment extends PreferenceFragment implements SharedPrefer
|
||||||
private final String HIFI_SETTINGS_AEC_KEY = "aec";
|
private final String HIFI_SETTINGS_AEC_KEY = "aec";
|
||||||
private final String PREFERENCE_KEY_AEC = "aec";
|
private final String PREFERENCE_KEY_AEC = "aec";
|
||||||
|
|
||||||
|
private final boolean DEFAULT_AEC_ENABLED = true;
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onCreate(@Nullable Bundle savedInstanceState) {
|
public void onCreate(@Nullable Bundle savedInstanceState) {
|
||||||
super.onCreate(savedInstanceState);
|
super.onCreate(savedInstanceState);
|
||||||
addPreferencesFromResource(R.xml.settings);
|
addPreferencesFromResource(R.xml.settings);
|
||||||
|
boolean aecAvailable = AcousticEchoCanceler.isAvailable();
|
||||||
|
PreferenceManager.setDefaultValues(getContext(), R.xml.settings, false);
|
||||||
|
|
||||||
if (!AcousticEchoCanceler.isAvailable()) {
|
if (!aecAvailable) {
|
||||||
getPreferenceScreen().getPreferenceManager().findPreference("aec").setEnabled(false);
|
findPreference(PREFERENCE_KEY_AEC).setEnabled(false);
|
||||||
|
updateHifiSetting(HIFI_SETTINGS_ANDROID_GROUP, HIFI_SETTINGS_AEC_KEY, false);
|
||||||
}
|
}
|
||||||
|
|
||||||
getPreferenceScreen().getSharedPreferences().edit().putBoolean(PREFERENCE_KEY_AEC,
|
getPreferenceScreen().getSharedPreferences().edit().putBoolean(PREFERENCE_KEY_AEC,
|
||||||
getHifiSettingBoolean(HIFI_SETTINGS_ANDROID_GROUP, HIFI_SETTINGS_AEC_KEY, false));
|
aecAvailable && getHifiSettingBoolean(HIFI_SETTINGS_ANDROID_GROUP, HIFI_SETTINGS_AEC_KEY, DEFAULT_AEC_ENABLED)).commit();
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public static SettingsFragment newInstance() {
|
public static SettingsFragment newInstance() {
|
||||||
|
@ -46,15 +52,13 @@ public class SettingsFragment extends PreferenceFragment implements SharedPrefer
|
||||||
public void onPause() {
|
public void onPause() {
|
||||||
super.onPause();
|
super.onPause();
|
||||||
getPreferenceScreen().getSharedPreferences().unregisterOnSharedPreferenceChangeListener(this);
|
getPreferenceScreen().getSharedPreferences().unregisterOnSharedPreferenceChangeListener(this);
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, String key) {
|
public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, String key) {
|
||||||
Preference pref = findPreference(key);
|
|
||||||
switch (key) {
|
switch (key) {
|
||||||
case "aec":
|
case PREFERENCE_KEY_AEC:
|
||||||
updateHifiSetting(HIFI_SETTINGS_ANDROID_GROUP, HIFI_SETTINGS_AEC_KEY, sharedPreferences.getBoolean(key, false));
|
updateHifiSetting(HIFI_SETTINGS_ANDROID_GROUP, HIFI_SETTINGS_AEC_KEY, sharedPreferences.getBoolean(key, DEFAULT_AEC_ENABLED));
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
|
|
|
@ -6,6 +6,7 @@
|
||||||
<SwitchPreference
|
<SwitchPreference
|
||||||
android:key="aec"
|
android:key="aec"
|
||||||
android:title="@string/AEC"
|
android:title="@string/AEC"
|
||||||
android:summary="@string/acoustic_echo_cancellation" />
|
android:summary="@string/acoustic_echo_cancellation"
|
||||||
|
android:defaultValue="true" />
|
||||||
</PreferenceCategory>
|
</PreferenceCategory>
|
||||||
</PreferenceScreen>
|
</PreferenceScreen>
|
|
@ -465,7 +465,7 @@ QAudioDeviceInfo defaultAudioDeviceForMode(QAudio::Mode mode) {
|
||||||
|
|
||||||
#if defined (Q_OS_ANDROID)
|
#if defined (Q_OS_ANDROID)
|
||||||
if (mode == QAudio::AudioInput) {
|
if (mode == QAudio::AudioInput) {
|
||||||
Setting::Handle<bool> enableAEC(SETTING_AEC_KEY, false);
|
Setting::Handle<bool> enableAEC(SETTING_AEC_KEY, DEFAULT_AEC_ENABLED);
|
||||||
bool aecEnabled = enableAEC.get();
|
bool aecEnabled = enableAEC.get();
|
||||||
auto audioClient = DependencyManager::get<AudioClient>();
|
auto audioClient = DependencyManager::get<AudioClient>();
|
||||||
bool headsetOn = audioClient? audioClient->isHeadsetPluggedIn() : false;
|
bool headsetOn = audioClient? audioClient->isHeadsetPluggedIn() : false;
|
||||||
|
@ -1673,7 +1673,7 @@ void AudioClient::setHeadsetPluggedIn(bool pluggedIn) {
|
||||||
QThread::msleep(200);
|
QThread::msleep(200);
|
||||||
}
|
}
|
||||||
|
|
||||||
Setting::Handle<bool> enableAEC(SETTING_AEC_KEY, false);
|
Setting::Handle<bool> enableAEC(SETTING_AEC_KEY, DEFAULT_AEC_ENABLED);
|
||||||
bool aecEnabled = enableAEC.get();
|
bool aecEnabled = enableAEC.get();
|
||||||
|
|
||||||
if ((pluggedIn || !aecEnabled) && _inputDeviceInfo.deviceName() != VOICE_RECOGNITION) {
|
if ((pluggedIn || !aecEnabled) && _inputDeviceInfo.deviceName() != VOICE_RECOGNITION) {
|
||||||
|
|
|
@ -69,6 +69,7 @@
|
||||||
#define VOICE_COMMUNICATION "voicecommunication"
|
#define VOICE_COMMUNICATION "voicecommunication"
|
||||||
|
|
||||||
#define SETTING_AEC_KEY "Android/aec"
|
#define SETTING_AEC_KEY "Android/aec"
|
||||||
|
#define DEFAULT_AEC_ENABLED true
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
class QAudioInput;
|
class QAudioInput;
|
||||||
|
|
Loading…
Reference in a new issue