Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Some changes in DirectInput sample#18

Open
DJm00n wants to merge 1 commit intowalbourn:mainfrom
DJm00n:master
Open

Some changes in DirectInput sample#18
DJm00n wants to merge 1 commit intowalbourn:mainfrom
DJm00n:master

Conversation

@DJm00n
Copy link

@DJm00n DJm00n commented Apr 9, 2020

  • Simplified XInput check - you can just get HID path via DIPROP_GUIDANDPATH and check for "_IG" there
  • Added DirectInput gamepad info printed on UI
  • Converted docx readme to Github friendly Markdown

- Simplified XInput check - you can just get HID path via DIPROP_GUIDANDPATH and check for "_IG" there
- Added DirectInput gamepad info printed on UI
- Converted docx readme to Github friendly Markdown
@DJm00n
Copy link
Author

DJm00n commented Apr 29, 2020

@walbourn some feedback on this?


[Where is the DirectX SDK (2013 Edition)?](http://blogs.msdn.com/b/chuckw/archive/2013/07/01/where-is-the-directx-sdk-2013-edition.aspx)

[Games for Windows and DirectX SDK blog](http://blogs.msdn.com/b/chuckw/) No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

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

UINT iDevice = 0;
VARIANT var;
HRESULT hr;
DIPROPGUIDANDPATH dip;
Copy link
Owner

Choose a reason for hiding this comment

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

Using DI directly instead of WMI does look a lot simpler. Have you been able to check this on more than one version of Windows?

Copy link
Author

@DJm00n DJm00n May 3, 2020

Choose a reason for hiding this comment

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

Unfortunately I only have Win10. Will try to install win 7 vm for testing xinput check.

@walbourn
Copy link
Owner

I've merged the walbourn/directx-sdk-samples and walbourn/directx-sdk-legacy-samples into https://github.com/microsoft/DirectX-SDK-Samples/.

I'm archiving this repo as all work should move there.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants