AndroidSystem delegate doesn't check for null audio renderers

[riccardobl]

The AndroidSystem doesn't seem to have a way to allow to disable the audio renderer, see code below:

jmonkeyengine/jme3-android/src/main/java/com/jme3/system/android/JmeAndroidSystem.java

Lines 79 to 86 in 09b862d

	if (settings.getAudioRenderer().equals(AppSettings.ANDROID_MEDIAPLAYER)) {
	audioRendererType = AppSettings.ANDROID_MEDIAPLAYER;
	} else if (settings.getAudioRenderer().equals(AppSettings.ANDROID_OPENAL_SOFT)) {
	audioRendererType = AppSettings.ANDROID_OPENAL_SOFT;
	} else {
	logger.log(Level.INFO, "AudioRenderer not set. Defaulting to OpenAL Soft");
	audioRendererType = AppSettings.ANDROID_OPENAL_SOFT;
	}

Setting audio renderer to null, as you would do with desktop, causes the engine to crash, due to the missing nullcheck.

[stephengold]

Thanks for documenting this issue, @riccardobl .

[github-actions]

💰 This issue has a bounty

Resolve it to receive a reward.
For details (amount, rules, eligibility), see: https://hub.jmonkeyengine.org/t/bounty-program-trial-starts-today/49394/

If you want to start working on this, comment on this issue with your intent.
If accepted by a maintainer, the issue will be assigned to you.

[quazi-irfan]

Does audioRendererType need to get set to AppSettings.ANDROID_OPENAL_SOFT when settings.getAudioRenderer() returns null?

[riccardobl]

i think the flow would be like:

settings.setAudioRenderer(null);

that turns audioRendererType to null
and disable all audio related stuff on jme3-android