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

EXI_Channel: Use std::array #4011

Merged
merged 2 commits into from Jul 15, 2016
Merged

EXI_Channel: Use std::array #4011

merged 2 commits into from Jul 15, 2016

Conversation

lioncash
Copy link
Member

@lioncash lioncash commented Jul 15, 2016 •
edited by delroth

Really basic change. Also drops Hungarian notation from the variable name.

This change is Reviewable

@lioncash
EXI_Channel: Use std::array
642284f
@lioncash
EXI_Channel: Change m_pDevices to m_devices
5b2ddbc 
Basic name change to get rid of (incorrect) Hungarian notation
@Parlane
Copy link
Member

Parlane commented Jul 15, 2016

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

Comments from Reviewable

@delroth delroth merged commit 7ca32f4 into dolphin-emu:master Jul 15, 2016
@lioncash lioncash deleted the array-exi branch July 15, 2016 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
3 participants
@lioncash @Parlane @delroth