Skip to content

Added Shell32.SHGetKnownFolderPath and corresponding constants#334

Merged
dblock merged 3 commits intojava-native-access:masterfrom
msteiger:knownfolder
May 26, 2014
Merged

Added Shell32.SHGetKnownFolderPath and corresponding constants#334
dblock merged 3 commits intojava-native-access:masterfrom
msteiger:knownfolder

Conversation

@msteiger
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wanted to use LPSTR.ByReference here, but the returned string was garbage. Can't really explain why ..

@dblock
Copy link
Copy Markdown
Member

dblock commented May 24, 2014

This is good. Check out my comments on CoTaskMemFree and the tests.

@msteiger
Copy link
Copy Markdown
Contributor Author

I ended up doing both: I changed the inheritance of LPVOID as discussed with @twall, but use plain Pointer instances which are more natural to work with.

@dblock
Copy link
Copy Markdown
Member

dblock commented May 26, 2014

That looks good.

@dblock
Copy link
Copy Markdown
Member

dblock commented May 26, 2014

Merging, nice work.

dblock added a commit that referenced this pull request May 26, 2014
Added Shell32.SHGetKnownFolderPath and corresponding constants
@dblock dblock merged commit 296e89e into java-native-access:master May 26, 2014
mstyura pushed a commit to mstyura/jna that referenced this pull request Sep 9, 2024
…ive-access#334)

Motivation:

The tree includes 'Maven Wrapper' with settings such that it downloads Maven 3.8.1 from the ASF 'backup' distribution servers, rather than the main distribution mirrors, or more typically in the wrappers case actually Maven Central (which it seems the settings did use prior to the last config update). As the CI jobs are grabbing Maven several times for every commit, plus adding on any related use from peoples Netty forks/downloads etc, this will add up. It would be good to direct them elsewhere.

Modifications:

Change url

Result:

Less load on the apache backup download server
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.

4 participants