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.

Sign up for GitHub

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

Jump to bottom

Common/IniFile: Move interface into Common namespace #11756

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Common/IniFile: Move interface into Common namespace #11756

merged 1 commit into from
Apr 14, 2023

Conversation

lioncash
Copy link
Member

Gets this out of the global namespace and into the Common namespace

@lioncash
Common/IniFile: Move interface into Common namespace
e4caace 
Gets this out of the global namespace and into the Common namespace
AdmiralCurtiss
AdmiralCurtiss approved these changes Apr 14, 2023
View reviewed changes
Copy link
Contributor

@AdmiralCurtiss AdmiralCurtiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine. I'm kinda shocked we have so many uses of this class...

@AdmiralCurtiss AdmiralCurtiss merged commit e9dbb93 into dolphin-emu:master Apr 14, 2023
@lioncash lioncash deleted the ini branch April 15, 2023 07:18
@Medard22 Medard22 mentioned this pull request Mar 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AdmiralCurtiss AdmiralCurtiss AdmiralCurtiss approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lioncash @AdmiralCurtiss