Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed fastmem from Android UI #3013

Merged
merged 1 commit into from Sep 11, 2015

Conversation

Helios747
Copy link
Contributor

@Sonicadvance1 recommended this.

@degasus
Copy link
Member

degasus commented Sep 9, 2015

I doubt this is enough. Can you search for "fastmemPref" in the Android directory?

@Helios747
Copy link
Contributor Author

Oh, do I need to remove the other references to fastmem in the Userpreferences.java file? I still want it to pick up whatever the user changes in the ini and my shield is refusing to write dolphin's config files to storage for some stupid reason so I literally can't test this.

@degasus
Copy link
Member

degasus commented Sep 9, 2015

Reading the ini is done in core, so you can remove fastmem everywhere android specific.

@@ -133,8 +132,6 @@ public static void SavePrefsToIni(Context ctx)
{
SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(ctx);

// Whether or not the user is using dual core.
boolean isUsingDualCore = prefs.getBoolean("dualCorePref", true);

This comment was marked as off-topic.

@waddlesplash
Copy link
Contributor

LGTM

Sonicadvance1 added a commit that referenced this pull request Sep 11, 2015
Removed fastmem from Android UI
@Sonicadvance1 Sonicadvance1 merged commit c319fbb into dolphin-emu:master Sep 11, 2015
@Helios747 Helios747 deleted the Security101 branch December 22, 2015 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants