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

Allow multiple GC adapters on one PC to be used with multiple Dolphin instances. #3539

Merged
merged 1 commit into from
Jan 20, 2016

Conversation

Sonicadvance1
Copy link
Contributor

Previously we would iterate through every GC adapter plugged in to the PC and steal ownership of it.
This causes issues all over the place in the implementation if this happens.
Break on the first adapter we can get access to.

Review on Reviewable

… instances.

Previously we would iterate through every GC adapter plugged in to the PC and steal ownership of it.
This causes issues all over the place in the implementation if this happens.
Break on the first adapter we can get access to.
@mathieui
Copy link
Member

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@delroth
Copy link
Member

delroth commented Jan 20, 2016

Reviewed 1 of 1 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

delroth added a commit that referenced this pull request Jan 20, 2016
Allow multiple GC adapters on one PC to be used with multiple Dolphin instances.
@delroth delroth merged commit 3a12dcc into dolphin-emu:master Jan 20, 2016
@Sonicadvance1 Sonicadvance1 deleted the two_adapters_one_pc branch January 20, 2016 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants