[fireball] 41279: Mikhail Denisenko <[e-mail]> - Split the PIDL into current and parent folder, and call GetFolderAttributes only for current folder PIDL. This fixes tons of "ERR: HCR_GetFolderAttributes should be called for simple PIDL's only!" debug log spam messages. See issue #4494 for more details.

Author: fireball
Date: Thu Jun  4 13:01:10 2009
New Revision: 41279

URL: http://svn.reactos.org/svn/reactos?rev=41279&view=rev
Log:
Mikhail Denisenko [e-mail]
- Split the PIDL into current and parent folder, and call GetFolderAttributes only for current folder PIDL. This fixes tons of "ERR: HCR_GetFolderAttributes should be called for simple PIDL's only!" debug log spam messages.

See issue #4494 for more details.

Modified:
    trunk/reactos/base/shell/explorer/shell/shellfs.cpp

Modified: trunk/reactos/base/shell/explorer/shell/shellfs.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/explorer/shell/s...
==============================================================================
--- trunk/reactos/base/shell/explorer/shell/shellfs.cpp [iso-8859-1] (original)
+++ trunk/reactos/base/shell/explorer/shell/shellfs.cpp [iso-8859-1] Thu Jun  4 13:01:10 2009
@@ -172,7 +172,11 @@

SFGAOF attribs = SFGAO_FILESYSTEM;

-	if (FAILED(const_cast<ShellFolder&>(_folder)->GetAttributesOf(1, (LPCITEMIDLIST*)&_pidl, &attribs)))
+	// Split pidl into current and parent folder PIDLs
+	ShellPath pidlParent, pidlFolder;
+	_pidl.split(pidlParent, pidlFolder);
+
+	if (FAILED(const_cast<ShellFolder&>(_folder)->GetAttributesOf(1, (LPCITEMIDLIST*)&pidlFolder, &attribs)))
    	return false;

if (!(attribs & SFGAO_FILESYSTEM))