[ros-diffs] [reactos] 01/01: [SHELL32] Improve Format dialog debug log (trivial) (#3176)

[Kyle Katarn]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=6d4b853b8a0df3401da87…

commit 6d4b853b8a0df3401da871ee2618f39e356328da
Author:     Kyle Katarn <contact(a)kcsoftwares.com>
AuthorDate: Sun Sep 13 23:05:04 2020 +0200
Commit:     GitHub <noreply(a)github.com>
CommitDate: Sun Sep 13 23:05:04 2020 +0200

    [SHELL32] Improve Format dialog debug log (trivial) (#3176)
    
    - Logging of name for unknown filesystem.
    - Fix some trace content as not only HDD could be affected.
---
 dll/win32/shell32/dialogs/drive.cpp | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/dll/win32/shell32/dialogs/drive.cpp b/dll/win32/shell32/dialogs/drive.cpp
index 60243d44a02..126dc0da211 100644
--- a/dll/win32/shell32/dialogs/drive.cpp
+++ b/dll/win32/shell32/dialogs/drive.cpp
@@ -212,7 +212,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     {
         if (!GetDefaultClusterSize(wszBuf, &ClusterSize, &TotalNumberOfBytes))
         {
-            TRACE("FAT16 is not supported on hdd larger than 4G current %lu\n",
TotalNumberOfBytes.QuadPart);
+            TRACE("FAT16 is not supported on drive larger than 4G current
%lu\n", TotalNumberOfBytes.QuadPart);
             SendMessageW(hDlgCtrl, CB_DELETESTRING, iSelIndex, 0);
             return;
         }
@@ -234,7 +234,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     {
         if (!GetDefaultClusterSize(wszBuf, &ClusterSize, &TotalNumberOfBytes))
         {
-            TRACE("FAT32 is not supported on hdd larger than 32G current
%lu\n", TotalNumberOfBytes.QuadPart);
+            TRACE("FAT32 is not supported on drive larger than 32G current
%lu\n", TotalNumberOfBytes.QuadPart);
             SendMessageW(hDlgCtrl, CB_DELETESTRING, iSelIndex, 0);
             return;
         }
@@ -256,7 +256,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     {
         if (!GetDefaultClusterSize(wszBuf, &ClusterSize, &TotalNumberOfBytes))
         {
-            TRACE("NTFS is not supported on hdd larger than 2TB current %lu\n",
TotalNumberOfBytes.QuadPart);
+            TRACE("NTFS is not supported on drive larger than 2TB current
%lu\n", TotalNumberOfBytes.QuadPart);
             SendMessageW(hDlgCtrl, CB_DELETESTRING, iSelIndex, 0);
             return;
         }
@@ -289,7 +289,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     {
         if (!GetDefaultClusterSize(wszBuf, &ClusterSize, &TotalNumberOfBytes))
         {
-            TRACE("EXT2 is not supported on hdd larger than 32T current %lu\n",
TotalNumberOfBytes.QuadPart);
+            TRACE("EXT2 is not supported on drive larger than 32T current
%lu\n", TotalNumberOfBytes.QuadPart);
             SendMessageW(hDlgCtrl, CB_DELETESTRING, iSelIndex, 0);
             return;
         }
@@ -310,7 +310,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     {
         if (!GetDefaultClusterSize(wszBuf, &ClusterSize, &TotalNumberOfBytes))
         {
-            TRACE("BtrFS is not supported on hdd larger than 16E current
%lu\n", TotalNumberOfBytes.QuadPart);
+            TRACE("BtrFS is not supported on drive larger than 16E current
%lu\n", TotalNumberOfBytes.QuadPart);
             SendMessageW(hDlgCtrl, CB_DELETESTRING, iSelIndex, 0);
             return;
         }
@@ -329,7 +329,7 @@ InsertDefaultClusterSizeForFs(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT
pContext)
     }
     else
     {
-        FIXME("unknown fs\n");
+        FIXME("Unknown filesystem: %ls\n", wszBuf);
         SendDlgItemMessageW(hwndDlg, 28680, CB_RESETCONTENT, iSelIndex, 0);
         return;
     }
@@ -444,7 +444,7 @@ FormatExCB(
         case FSNOTSUPPORTED:
         case CLUSTERSIZETOOSMALL:
             bSuccess = FALSE;
-            FIXME("\n");
+            FIXME("Unsupported command in FormatExCB\n");
             break;
 
         default:
@@ -476,13 +476,13 @@ FormatDrive(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT pContext)
     iSelIndex = SendMessageW(hDlgCtrl, CB_GETCURSEL, 0, 0);
     if (iSelIndex == CB_ERR)
     {
-        FIXME("\n");
+        ERR("Unable to get file system selection\n");
         return;
     }
     Length = SendMessageW(hDlgCtrl, CB_GETLBTEXTLEN, iSelIndex, 0);
     if ((int)Length == CB_ERR || Length + 1 > _countof(szFileSys))
     {
-        FIXME("\n");
+        ERR("Unable to get file system selection\n");
         return;
     }
 
@@ -495,7 +495,7 @@ FormatDrive(HWND hwndDlg, PFORMAT_DRIVE_CONTEXT pContext)
     Length = SendMessageW(hDlgCtrl, WM_GETTEXTLENGTH, 0, 0);
     if (Length + 1 > _countof(szLabel))
     {
-        FIXME("\n");
+        ERR("Unable to get volume label\n");
         return;
     }
     SendMessageW(hDlgCtrl, WM_GETTEXT, _countof(szLabel), (LPARAM)szLabel);