Hi John, a few problems with this one:
-{ +{ This whitespace-only change isn't needed for your patch, please omit it.
+ if (info->pidlRet == NULL) + return FALSE; + { This is poor style, IMHO. Surely an explicit else would be better than the implicit else block here?
+ MessageBoxA( info->hwndTreeView, + "You cannot create a new folder here. Choose a different location.", + "Invalid location", MB_OK ); The indentation is off, please match the surrounding code. And, these should be in resources, not in English in the source.
--Juan