Windows Explorer Shell Extension: Remove OutputDebugString calls

Remove all OutputDebugString calls from the explorer shell extension.
OutputDebugString() requires acquiring system global locks.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/643
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
This commit is contained in:
Jeffrey Altman 2009-10-11 18:06:25 -04:00 committed by Jeffrey Altman
parent 93fb21a8d0
commit e026fcb2f0

View File

@ -1166,11 +1166,6 @@ BOOL IsPathInAfs(const CString & strPath)
HOURGLASS hourglass;
CString debugBuf;
debugBuf.Format(_T("IsPathInAfs(%s)"), strPath);
OutputDebugString(debugBuf);
memset(&options, 0, sizeof(options));
options.size = sizeof(options);
options.field_flags |= CM_IOCTL_QOPTS_FIELD_LITERAL;
@ -1181,10 +1176,6 @@ BOOL IsPathInAfs(const CString & strPath)
blob.out = (char *) &fid;
code = pioctl_T(strPath, VIOCGETFID, &blob, 1);
debugBuf.Format(_T("VIOCGETFID=%d"), code);
OutputDebugString(debugBuf);
if (code) {
if ((errno == EINVAL) || (errno == ENOENT))
return FALSE;
@ -1642,12 +1633,6 @@ BOOL IsSymlink(const CString& strName)
HOURGLASS hourglass;
{
CString str;
str.Format(_T("IsSymlink(%s)"), strName);
OutputDebugString(str);
}
CStringUtf8 ustrLast(LastComponent(strName));
CString strParent = Parent(strName);
@ -1675,27 +1660,11 @@ BOOL IsMountPoint(const CString& path)
HOURGLASS hourglass;
{
CString str;
str.Format(_T("IsMountPoint(%s)"), path);
OutputDebugString(str);
}
CString parent = Parent(path);
FixNetbiosPath(parent);
CStringUtf8 mountpoint(LastComponent(path));
{
CString str;
#ifdef UNICODE
str.Format(_T("last_component=%S"), mountpoint);
#else
str.Format(_T("last_component=%s"), mountpoint);
#endif
OutputDebugString(str);
}
blob.in_size = mountpoint.GetLength() + 1;
blob.in = mountpoint.GetBuffer();
blob.out = lsbuffer;
@ -2052,13 +2021,6 @@ UINT MakeSymbolicLink(const CString& strName, const CString& strTarget)
CString strParent = Parent(strName);
FixNetbiosPath(strParent);
{
CString str;
str.Format(_T("MakeSymbolicLink: name = %s target = %s parent = %s\n"),
strName, strTarget, strParent);
OutputDebugString(str);
}
if ( IsFreelanceRoot(strParent) && !IsAdmin() ) {
ShowMessageBox(IDS_NOT_AFS_CLIENT_ADMIN_ERROR, MB_ICONERROR, IDS_NOT_AFS_CLIENT_ADMIN_ERROR);
return FALSE;