Windows Explorer Shell Extension: Remove OutputDebugString calls
authorJeffrey Altman <jaltman@secure-endpoints.com>
Sun, 11 Oct 2009 22:06:25 +0000 (18:06 -0400)
committerJeffrey Altman <jaltman|account-1000011@unknown>
Mon, 12 Oct 2009 14:28:26 +0000 (07:28 -0700)
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>

src/WINNT/client_exp/gui2fs.cpp

index c5be1c0..0c14029 100644 (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;