Skip to content
Permalink
Browse files

Android: Suggest deleting game INIs if they contain global INI data

  • Loading branch information...
JosJuice committed Jun 18, 2019
1 parent 7652023 commit 7f841e9bfd6dad709335ab7937fa03279d47adb4
@@ -253,6 +253,8 @@ public static void rumble(int padID, double state)
Rumble.checkRumble(padID, state);
}

public static native void NewGameIniFile();

public static native void LoadGameIniFile(String gameId);

public static native void SaveGameIniFile(String gameId);
@@ -184,4 +184,36 @@ public void saveSettings(SettingsActivityView view)
SettingsFile.saveCustomGameSettings(gameId, sections);
}
}

public void clearSettings()
{
sections.clear();
}

public boolean gameIniContainsJunk()
{
// Older versions of Android Dolphin would copy the entire contents of most of the global INIs
// into any game INI that got saved (with some of the sections renamed to match the game INI
// section names). The problems with this are twofold:
//
// 1. The user game INIs will contain entries that Dolphin doesn't support reading from
// game INIs. This is annoying when editing game INIs manually but shouldn't really be
// a problem for those who only use the GUI.
//
// 2. Global settings will stick around in user game INIs. For instance, if someone wants to
// change the texture cache accuracy to safe for all games, they have to edit not only the
// global settings but also every single game INI they have created, since the old value of
// the texture cache accuracy setting has been copied into every user game INI.
//
// These problems are serious enough that we should detect and delete such INI files.
// Problem 1 is easy to detect, but due to the nature of problem 2, it's unfortunately not
// possible to know which lines were added intentionally by the user and which lines were added
// unintentionally, which is why we have to delete the whole file in order to fix everything.

if (TextUtils.isEmpty(gameId))
return false;

SettingSection interfaceSection = sections.get("Interface");
return interfaceSection != null && interfaceSection.getSetting("ThemeName") != null;
}
}
@@ -1,5 +1,6 @@
package org.dolphinemu.dolphinemu.features.settings.ui;

import android.app.AlertDialog;
import android.app.ProgressDialog;
import android.content.Context;
import android.content.Intent;
@@ -184,6 +185,18 @@ public void showExternalStorageNotMountedHint()
.show();
}

@Override
public void showGameIniJunkDeletionQuestion()
{
new AlertDialog.Builder(this)
.setTitle(getString(R.string.game_ini_junk_title))
.setMessage(getString(R.string.game_ini_junk_question))
.setPositiveButton(R.string.yes, (dialogInterface, i) -> mPresenter.clearSettings())
.setNegativeButton(R.string.no, null)
.create()
.show();
}

@Override
public org.dolphinemu.dolphinemu.features.settings.model.Settings getSettings()
{
@@ -58,6 +58,11 @@ private void loadSettingsUI()
if (!TextUtils.isEmpty(gameId))
{
mSettings.loadSettings(gameId, mView);

if (mSettings.gameIniContainsJunk())
{
mView.showGameIniJunkDeletionQuestion();
}
}
else
{
@@ -118,6 +123,12 @@ public Settings getSettings()
return mSettings;
}

public void clearSettings()
{
mSettings.clearSettings();
onSettingChanged();
}

public void onStop(boolean finishing)
{
if (directoryStateReceiver != null)
@@ -119,6 +119,11 @@
*/
void showExternalStorageNotMountedHint();

/**
* Tell the user that there is junk in the game INI and ask if they want to delete the whole file.
*/
void showGameIniJunkDeletionQuestion();

/**
* Start the DirectoryInitialization and listen for the result.
*
@@ -452,7 +452,8 @@ public static void saveCustomGameSettings(final String gameId,
final HashMap<String, SettingSection> sections)
{
Set<String> sortedSections = new TreeSet<>(sections.keySet());
NativeLibrary.LoadGameIniFile(gameId);

NativeLibrary.NewGameIniFile();
for (String sectionKey : sortedSections)
{
SettingSection section = sections.get(sectionKey);
@@ -274,6 +274,8 @@
<string name="preferences_settings">Settings</string>
<string name="preferences_game_settings">Game Settings</string>
<string name="preferences_extensions">Extension Bindings</string>
<string name="game_ini_junk_title">Junk Data Found</string>
<string name="game_ini_junk_question">The settings file for this game contains junk data created by an old version of Dolphin. Would you like to fix this by deleting the settings file for this game? This cannot be undone.</string>

<!-- Emulation Menu -->
<string name="emulation_screenshot">Take Screenshot</string>
@@ -357,6 +357,12 @@ JNIEXPORT jstring JNICALL Java_org_dolphinemu_dolphinemu_NativeLibrary_GetUserSe
return ToJString(env, value.c_str());
}

JNIEXPORT void JNICALL Java_org_dolphinemu_dolphinemu_NativeLibrary_NewGameIniFile(JNIEnv* env,
jobject obj)
{
s_ini = IniFile();
}

JNIEXPORT void JNICALL Java_org_dolphinemu_dolphinemu_NativeLibrary_LoadGameIniFile(JNIEnv* env,
jobject obj,
jstring jGameID)

0 comments on commit 7f841e9

Please sign in to comment.
You can’t perform that action at this time.