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

VideoConfig: Add const specifier to IsVSync() member function #6184

Merged
merged 1 commit into from Nov 11, 2017

Conversation

3 participants
@lioncash
Member

lioncash commented Nov 11, 2017

This member function doesn't alter VideoConfig's state.

Pretty easy PR to review.

VideoConfig: Add const specifier to IsVSync() member function
This member function doesn't alter VideoConfig's state.
@ligfx

ligfx approved these changes Nov 11, 2017

@Helios747

This comment has been minimized.

Show comment
Hide comment
@Helios747

Helios747 Nov 11, 2017

Contributor

set my computer on fire. Thanks. Now I'm mad so I'm going to set everybody else's computers on fire.

Contributor

Helios747 commented Nov 11, 2017

set my computer on fire. Thanks. Now I'm mad so I'm going to set everybody else's computers on fire.

@Helios747 Helios747 merged commit 6161bda into dolphin-emu:master Nov 11, 2017

10 checks passed

default Very basic checks passed, handed off to Buildbot.
Details
lint Build succeeded on builder lint
Details
pr-android Build succeeded on builder pr-android
Details
pr-deb-dbg-x64 Build succeeded on builder pr-deb-dbg-x64
Details
pr-deb-x64 Build succeeded on builder pr-deb-x64
Details
pr-freebsd-x64 Build succeeded on builder pr-freebsd-x64
Details
pr-osx-x64 Build succeeded on builder pr-osx-x64
Details
pr-ubu-x64 Build succeeded on builder pr-ubu-x64
Details
pr-win-dbg-x64 Build succeeded on builder pr-win-dbg-x64
Details
pr-win-x64 Build succeeded on builder pr-win-x64
Details

@lioncash lioncash deleted the lioncash:const branch Nov 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment