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 unused usings. #3593

Merged
merged 3 commits into from
Aug 18, 2022
Merged

Conversation

CloneDeath
Copy link
Contributor

As the title says, I just removed unused usings.
There is still an unused using in the generated SPIRV file (Vulkan). But I think we can deal with that at a future date, separately.

@gdkchan gdkchan added enhancement New feature or request miscellaneous Related to some project not listed in labels labels Aug 16, 2022
Copy link
Member

@gdkchan gdkchan left a comment

Choose a reason for hiding this comment

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

lgtm, thanks.

Copy link
Contributor

@marysaka marysaka left a comment

Choose a reason for hiding this comment

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

A long overdue cleanup, great work 👍

@marysaka
Copy link
Contributor

Will need a rebase before merge

@CloneDeath
Copy link
Contributor Author

@Thog rebased.

@marysaka marysaka merged commit 951700f into Ryujinx:master Aug 18, 2022
@CloneDeath CloneDeath deleted the unused-using-removal branch August 18, 2022 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request miscellaneous Related to some project not listed in labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants