Michael Stefaniuc : kernelbase: Use wide-char string literals.

Alexandre Julliard julliard at winehq.org
Thu Nov 26 16:30:59 CST 2020


Module: wine
Branch: master
Commit: 1b1b6c5f7d28768feb853eefd015aa714b819ed4
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=1b1b6c5f7d28768feb853eefd015aa714b819ed4

Author: Michael Stefaniuc <mstefani at winehq.org>
Date:   Wed Nov 25 23:52:03 2020 +0100

kernelbase: Use wide-char string literals.

Signed-off-by: Michael Stefaniuc <mstefani at winehq.org>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/kernelbase/path.c       |  2 +-
 dlls/kernelbase/registry.c   |  3 +--
 dlls/kernelbase/tests/path.c | 12 ++++++------
 dlls/kernelbase/volume.c     | 10 ++++------
 4 files changed, 12 insertions(+), 15 deletions(-)

diff --git a/dlls/kernelbase/path.c b/dlls/kernelbase/path.c
index d7e2f0a2347..f0c79bcc94c 100644
--- a/dlls/kernelbase/path.c
+++ b/dlls/kernelbase/path.c
@@ -2944,7 +2944,7 @@ HRESULT WINAPI UrlUnescapeW(WCHAR *url, WCHAR *unescaped, DWORD *unescaped_len,
         else if (*src == '%' && isxdigit(*(src + 1)) && isxdigit(*(src + 2)) && !stop_unescaping)
         {
             INT ih;
-            WCHAR buf[5] = {'0','x',0};
+            WCHAR buf[5] = L"0x";
             memcpy(buf + 2, src + 1, 2*sizeof(WCHAR));
             buf[4] = 0;
             StrToIntExW(buf, STIF_SUPPORT_HEX, &ih);
diff --git a/dlls/kernelbase/registry.c b/dlls/kernelbase/registry.c
index d1588d0b265..574bf2f56b7 100644
--- a/dlls/kernelbase/registry.c
+++ b/dlls/kernelbase/registry.c
@@ -2775,7 +2775,6 @@ LSTATUS WINAPI RegLoadMUIStringW(HKEY hKey, LPCWSTR pwszValue, LPWSTR pwszBuffer
     /* Parse the value and load the string. */
     {
         WCHAR *pComma = wcsrchr(pwszExpandedBuffer, ','), *pNewBuffer;
-        const WCHAR backslashW[] = {'\\',0};
         UINT uiStringId;
         DWORD baseDirLen;
         int reqChars;
@@ -2802,7 +2801,7 @@ LSTATUS WINAPI RegLoadMUIStringW(HKEY hKey, LPCWSTR pwszValue, LPWSTR pwszBuffer
         if (baseDirLen) {
             lstrcpyW(pwszTempBuffer, pwszBaseDir);
             if (pwszBaseDir[baseDirLen - 1] != '\\')
-                lstrcatW(pwszTempBuffer, backslashW);
+                lstrcatW(pwszTempBuffer, L"\\");
         }
         lstrcatW(pwszTempBuffer, pwszExpandedBuffer + 1);
 
diff --git a/dlls/kernelbase/tests/path.c b/dlls/kernelbase/tests/path.c
index 2ee7b512b31..8d85542612f 100644
--- a/dlls/kernelbase/tests/path.c
+++ b/dlls/kernelbase/tests/path.c
@@ -546,9 +546,9 @@ static void test_PathAllocCombine(void)
 
 static void test_PathCchCombine(void)
 {
-    WCHAR expected[PATHCCH_MAX_CCH] = {'C', ':', '\\', 'a', 0};
-    WCHAR p1[PATHCCH_MAX_CCH] = {'C', ':', '\\', 0};
-    WCHAR p2[PATHCCH_MAX_CCH] = {'a', 0};
+    WCHAR expected[PATHCCH_MAX_CCH] = L"C:\\a";
+    WCHAR p1[PATHCCH_MAX_CCH] = L"C:\\";
+    WCHAR p2[PATHCCH_MAX_CCH] = L"a";
     WCHAR output[PATHCCH_MAX_CCH];
     HRESULT hr;
     INT i;
@@ -607,9 +607,9 @@ static void test_PathCchCombine(void)
 
 static void test_PathCchCombineEx(void)
 {
-    WCHAR expected[MAX_PATH] = {'C',':','\\','a',0};
-    WCHAR p1[MAX_PATH] = {'C',':','\\',0};
-    WCHAR p2[MAX_PATH] = {'a',0};
+    WCHAR expected[MAX_PATH] = L"C:\\a";
+    WCHAR p1[MAX_PATH] = L"C:\\";
+    WCHAR p2[MAX_PATH] = L"a";
     WCHAR output[MAX_PATH];
     HRESULT hr;
     int i;
diff --git a/dlls/kernelbase/volume.c b/dlls/kernelbase/volume.c
index 53cc0d49b65..7daf9c98823 100644
--- a/dlls/kernelbase/volume.c
+++ b/dlls/kernelbase/volume.c
@@ -152,7 +152,6 @@ static DWORD get_mountmgr_drive_type( LPCWSTR root )
 /* get the label by reading it from a file at the root of the filesystem */
 static void get_filesystem_label( const UNICODE_STRING *device, WCHAR *label, DWORD len )
 {
-    static const WCHAR labelW[] = {'.','w','i','n','d','o','w','s','-','l','a','b','e','l',0};
     HANDLE handle;
     UNICODE_STRING name;
     IO_STATUS_BLOCK io;
@@ -167,12 +166,12 @@ static void get_filesystem_label( const UNICODE_STRING *device, WCHAR *label, DW
     attr.SecurityDescriptor = NULL;
     attr.SecurityQualityOfService = NULL;
 
-    name.MaximumLength = device->Length + sizeof(labelW);
+    name.MaximumLength = device->Length + sizeof(L".windows-label");
     name.Length = name.MaximumLength - sizeof(WCHAR);
     if (!(name.Buffer = HeapAlloc( GetProcessHeap(), 0, name.MaximumLength ))) return;
 
     memcpy( name.Buffer, device->Buffer, device->Length );
-    memcpy( name.Buffer + device->Length / sizeof(WCHAR), labelW, sizeof(labelW) );
+    memcpy( name.Buffer + device->Length / sizeof(WCHAR), L".windows-label", sizeof(L".windows-label") );
     if (!NtOpenFile( &handle, GENERIC_READ | SYNCHRONIZE, &attr, &io, FILE_SHARE_READ|FILE_SHARE_WRITE,
                      FILE_NON_DIRECTORY_FILE | FILE_SYNCHRONOUS_IO_NONALERT ))
     {
@@ -193,7 +192,6 @@ static void get_filesystem_label( const UNICODE_STRING *device, WCHAR *label, DW
 /* get the serial number by reading it from a file at the root of the filesystem */
 static DWORD get_filesystem_serial( const UNICODE_STRING *device )
 {
-    static const WCHAR serialW[] = {'.','w','i','n','d','o','w','s','-','s','e','r','i','a','l',0};
     HANDLE handle;
     UNICODE_STRING name;
     IO_STATUS_BLOCK io;
@@ -207,12 +205,12 @@ static DWORD get_filesystem_serial( const UNICODE_STRING *device )
     attr.SecurityDescriptor = NULL;
     attr.SecurityQualityOfService = NULL;
 
-    name.MaximumLength = device->Length + sizeof(serialW);
+    name.MaximumLength = device->Length + sizeof(L".windows-serial");
     name.Length = name.MaximumLength - sizeof(WCHAR);
     if (!(name.Buffer = HeapAlloc( GetProcessHeap(), 0, name.MaximumLength ))) return 0;
 
     memcpy( name.Buffer, device->Buffer, device->Length );
-    memcpy( name.Buffer + device->Length / sizeof(WCHAR), serialW, sizeof(serialW) );
+    memcpy( name.Buffer + device->Length / sizeof(WCHAR), L".windows-serial", sizeof(L".windows-serial") );
     if (!NtOpenFile( &handle, GENERIC_READ | SYNCHRONIZE, &attr, &io, FILE_SHARE_READ|FILE_SHARE_WRITE,
                      FILE_SYNCHRONOUS_IO_NONALERT ))
     {




More information about the wine-cvs mailing list