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

windows: remove now-extraneous NOMINMAX and WIN32_LEAN_AND_MEAN #defines #857

Merged
merged 1 commit into from Aug 23, 2014
Merged

windows: remove now-extraneous NOMINMAX and WIN32_LEAN_AND_MEAN #defines #857

merged 1 commit into from Aug 23, 2014

Conversation

shuffle2
Copy link
Contributor

Wrap dinput.h in a header defining DIRECTINPUT_VERSION instead of repeating it multiple places.

if (FAILED(DirectInput8Create(GetModuleHandle(nullptr), DIRECTINPUT_VERSION, IID_IDirectInput8, (LPVOID*)&idi8, nullptr)))
if (FAILED(DirectInput8Create(GetModuleHandle(nullptr), DIRECTINPUT_VERSION,
IID_IDirectInput8, (LPVOID*)&idi8, nullptr)))
{

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@lioncash
Copy link
Member

Looks good to me.

@delroth
Copy link
Member

delroth commented Aug 23, 2014

LGTM.

@delroth
Copy link
Member

delroth commented Aug 23, 2014

Actually, could you just edit this PR title? It's truncated.

@shuffle2 shuffle2 changed the title windows: remove now-extraneous NOMINMAX and WIN32_LEAN_AND_MEAN #defines... windows: remove now-extraneous NOMINMAX and WIN32_LEAN_AND_MEAN #defines Aug 23, 2014
…nes from dolphin code.

Wrap dinput.h in a header defining DIRECTINPUT_VERSION instead of repeating it multiple places.
shuffle2 added a commit that referenced this pull request Aug 23, 2014
windows: remove now-extraneous NOMINMAX and WIN32_LEAN_AND_MEAN #defines
@shuffle2 shuffle2 merged commit f964a28 into dolphin-emu:master Aug 23, 2014
@shuffle2 shuffle2 deleted the msvc-improvements branch August 23, 2014 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants