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

Advapi32#RegNotifyChangeKeyValue should be supported #1338

Closed
Dani-Hub opened this issue Apr 10, 2021 · 1 comment · Fixed by #1339
Closed

Advapi32#RegNotifyChangeKeyValue should be supported #1338

Dani-Hub opened this issue Apr 10, 2021 · 1 comment · Fixed by #1339

Comments

@Dani-Hub
Copy link
Contributor

  1. Version of JNA and related jars: 5.8.0
  2. Version and vendor of the java virtual machine: Java-11 AdoptOpenJDK
  3. Operating system: Windows 10
  4. System architecture (CPU type, bitness of the JVM): JVM: 64-bit, processor Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz, 3301 MHz, 4 Kern(e), 4 logische(r) Prozessor(en)
  5. Complete description of the problem:
    Attempting to call the Advapi32 function RegNotifyChangeKeyValue (as specified here) doesn't work, because this method is not declared yet in interface Advapi32.
  6. Steps to reproduce: Always reproducible

A pull request is ongoing
Dani-Hub added a commit to Dani-Hub/jna that referenced this issue Apr 10, 2021
@Dani-Hub
java-native-access#1338 Advapi32#RegNotifyChangeKeyValue should be su…
c0038d0 
…pported

Attempting to call the Advapi32 function RegNotifyChangeKeyValue doesn't work, because this method is not declared yet in interface Advapi32. This adds the missing method declaration.
Dani-Hub added a commit to Dani-Hub/jna that referenced this issue Apr 10, 2021
@Dani-Hub
java-native-access#1338 Advapi32#RegNotifyChangeKeyValue should be su…
f0716b1 
…pported

This adds another unit test to Advapi32Test validating basic functionality of Advapi32#RegNotifyChangeKeyValue.
@Dani-Hub Dani-Hub mentioned this issue Apr 10, 2021
Merged
Dani-Hub added a commit to Dani-Hub/jna that referenced this issue Apr 10, 2021
@Dani-Hub
java-native-access#1338 Advapi32#RegNotifyChangeKeyValue should be su…
2674bc3 
…pported

Fixing check style issues
@Dani-Hub
Copy link
Contributor Author

Dani-Hub commented Apr 10, 2021 •
edited

Pull request 1339 attempts to solve the reported problem.

@dbwiddis dbwiddis linked a pull request Apr 15, 2021 that will close this issue
#1338 advapi32#reg notify change key value should be supported #1339
Merged
dbwiddis added a commit to Dani-Hub/jna that referenced this issue Apr 18, 2021
@dbwiddis
Merge branch 'master' into java-native-access#1338-Advapi32#RegNotify…
2d2e7b8 
…ChangeKeyValue-should-be-supported
dbwiddis added a commit to Dani-Hub/jna that referenced this issue Apr 18, 2021
@dbwiddis
Merge branch 'master' into java-native-access#1338-Advapi32#RegNotify…
3ad2a85 
…ChangeKeyValue-should-be-supported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#1338 advapi32#reg notify change key value should be supported Dani-Hub/jna
1 participant
@Dani-Hub