You can make the correction, I think it’s appropriate for your patch.  It’ll fix your “initialize from pointer” issue, although it does introduce an incompatible binary change if someone happens to be doing anything with LPVOID other than passing it around.

On May 25, 2014, at 8:10 AM, Martin Steiger <notifications@github.com> wrote:

> In contrib/platform/test/com/sun/jna/platform/win32/Shell32Test.java:
> 
> > @@ -127,4 +131,16 @@ public void testResizeDesktopFromTop() throws InterruptedException {
> >  
> >      }
> >  
> > +    public void testSHGetKnownFolderPath()
> > +    {
> > +        int flags = ShlObj.KNOWN_FOLDER_FLAG.NONE.getFlag();
> > +        PointerByReference outPath = new PointerByReference();
> > +        HANDLE token = null;
> > +        GUID guid = KnownFolders.FOLDERID_Fonts;
> > +        HRESULT hr = Shell32.INSTANCE.SHGetKnownFolderPath(guid, flags, token, outPath);
> > +
> > +        Ole32.INSTANCE.CoTaskMemFree(new LPVOID(outPath.getPointer().getLong(0)));
> 
> Do you want me to change that or prefer doing that yourself?
> 
> —
> Reply to this email directly or view it on GitHub.
>