Skip to content

Added 'SetupDiOpenDevRegKey' and 'SetupDiEnumDeviceInfo' to SetupApi.#419

Closed
ChristianSchwarz wants to merge 1 commit intojava-native-access:masterfrom
ChristianSchwarz:setup-api
Closed

Added 'SetupDiOpenDevRegKey' and 'SetupDiEnumDeviceInfo' to SetupApi.#419
ChristianSchwarz wants to merge 1 commit intojava-native-access:masterfrom
ChristianSchwarz:setup-api

Conversation

@ChristianSchwarz
Copy link
Copy Markdown

Added the methods 'SetupDiOpenDevRegKey' and 'SetupDiEnumDeviceInfo' and constants associated with it to the SetupApi.

NOTE: this pull request is a replacement for #335 since i failed to rebase it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So much space! Please cleanup.

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 1, 2015

Thanks. Take a look at the diff on Github, you have a problem with CR/LF vs. LF/CR it looks like, the diff should look clean here. Same with tabs vs. spaces. I know it's annoying, but we want history to look readable. Once that's fixed, please squash these commits - lmk if you need any help.

@ChristianSchwarz
Copy link
Copy Markdown
Author

I am unable to solve the CR/LF vs LF/CR issue, i reverted the CHANGES.md to the version of the last commit before my change but it still doesn't change back and shows diffs. In my Eclipse diff i dont see that at all, only my added line.
Any ideas?

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 1, 2015

Actually revert it and edit it with something else than Eclipse? I am not sure what to do, but I can merge this manually later.

@ChristianSchwarz
Copy link
Copy Markdown
Author

Yes i tried to revert it and change it in Notepad++, no luck. I can't even revert the CHANGES.md, the diffs are shown her in github, maybe this a github-bug?

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 1, 2015

Definitely not a Github bug :)

@ChristianSchwarz
Copy link
Copy Markdown
Author

Okay i have no luck with this CRLF LFCR issue. I tried other editors, git config core.autocrlf and manual configuration of the .gitattributes file and even convert the delimters in eclipse using file->convert delimiters-> Unix (LF) / Windows (CRLF).

Can you please add this to the CHANGES.md ->
* [#419](https://github.com/twall/jna/pull/419): Added SetupDiOpenDevRegKey,SetupDiEnumDeviceInfoand related constants tocom.sun.jna.platform.win32.SetupApi - [@ChristianSchwarz](https://github.com/ChristianSchwarz)

Out of interest, what delimiter is currently used for the CHANGES.md?

  • Unix LF
  • Win CRLF
  • Mac CR

constants to `com.sun.jna.platform.win32.SetupApi`
@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 6, 2015

@dblock
Copy link
Copy Markdown
Member

dblock commented Apr 6, 2015

Merged via 0b47d1e. Thanks for hanging in here!

@dblock dblock closed this Apr 6, 2015
@ChristianSchwarz
Copy link
Copy Markdown
Author

I used the secret magic of the mighty -> github-file-editor ;-)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants