🚀 go-pugleaf

Hi,

I'm using GetDisplayNameOf() to obtain the (localized) display name of 
the MyDocuments folder (i.e. CSIDL_PERSONAL).

This was somewhat confusing to implement, but I managed to get it 
working properly. However, I am unsure if I'm doing a proper clean-up 
after I'm finished ?

I'm especially concerned about the STRRET info at 
http://msdn2.microsoft.com/en-us/library/bb759820(VS.85).aspx that states:

pOleStr:
A pointer to the string. This memory must be allocated with 
CoTaskMemAlloc. It is the calling application's responsibility to free 
this memory with CoTaskMemFree when it is no longer needed.

Do I have to take care of freeing that memory, or is someone else taking 
care of freeing the STRRET.pOleStr (e.g. pDesktopFolder) ?

Please feel free to mention any other (clean-up) issues that you might 
notice with the code sample below.

Thanks,

Kris

CoInitialize(0) ;
LPITEMIDLIST pidl;
STRRET sName;
LPSHELLFOLDER pDesktopFolder;
if(SUCCEEDED(SHGetSpecialFolderLocation(NULL,CSIDL_PERSONAL,&pidl)))
{
if(SUCCEEDED(SHGetDesktopFolder(&pDesktopFolder)))
{
if(SUCCEEDED(pDesktopFolder->GetDisplayNameOf(pidl,SHGDN_INFOLDER,&sName)))
{
// Probably safe to assume STRRET_WSTR is set in this specific case
// So simply use the pOleStr member without any additional checks
WideString DisplayName=sName.pOleStr;
DEBUG("DisplayNameof(CSIDL_PERSONAL)=[%s]",String(DisplayName).c_str());
}
pDesktopFolder->Release();
}
CoTaskMemFree((void*)pidl);
}
CoUninitialize();