Dmitry Timoshkov : shell32/tests: Remove no longer used parameter.

Alexandre Julliard julliard at winehq.org
Wed Dec 23 15:33:16 CST 2020


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

Author: Dmitry Timoshkov <dmitry at baikal.ru>
Date:   Wed Dec 23 17:02:32 2020 +0300

shell32/tests: Remove no longer used parameter.

Signed-off-by: Dmitry Timoshkov <dmitry at baikal.ru>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/shell32/tests/shell32_test.h |  4 ++--
 dlls/shell32/tests/shelllink.c    | 23 +++++++++++------------
 dlls/shell32/tests/shlexec.c      |  4 ++--
 3 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/dlls/shell32/tests/shell32_test.h b/dlls/shell32/tests/shell32_test.h
index 42a74fb2d5f..89d08e70c0f 100644
--- a/dlls/shell32/tests/shell32_test.h
+++ b/dlls/shell32/tests/shell32_test.h
@@ -33,5 +33,5 @@ typedef struct
     WORD  hotkey;
 } lnk_desc_t;
 
-#define create_lnk(a,b,c)     create_lnk_(__LINE__, (a), (b), (c))
-void create_lnk_(int,const WCHAR*,lnk_desc_t*,int);
+#define create_lnk(a,b)     create_lnk_(__LINE__, (a), (b))
+void create_lnk_(int,const WCHAR*,lnk_desc_t*);
diff --git a/dlls/shell32/tests/shelllink.c b/dlls/shell32/tests/shelllink.c
index e4f176237f4..4f650d801d0 100644
--- a/dlls/shell32/tests/shelllink.c
+++ b/dlls/shell32/tests/shelllink.c
@@ -386,7 +386,7 @@ static void test_get_set(void)
 #define lok                   ok_(__FILE__, line)
 #define check_lnk(a,b,c)        check_lnk_(__LINE__, (a), (b), (c))
 
-void create_lnk_(int line, const WCHAR* path, lnk_desc_t* desc, int save_fails)
+void create_lnk_(int line, const WCHAR* path, lnk_desc_t* desc)
 {
     HRESULT r;
     IShellLinkA *sl;
@@ -460,8 +460,7 @@ void create_lnk_(int line, const WCHAR* path, lnk_desc_t* desc, int save_fails)
         lok(str == NULL, "got %p\n", str);
 
         r = IPersistFile_Save(pf, path, TRUE);
-        todo_wine_if (save_fails)
-            lok(r == S_OK, "save failed (0x%08x)\n", r);
+        lok(r == S_OK, "save failed (0x%08x)\n", r);
 
         /* test GetCurFile after ::Save */
         r = IPersistFile_GetCurFile(pf, &str);
@@ -628,7 +627,7 @@ static void test_load_save(void)
 
     /* Save an empty .lnk file */
     memset(&desc, 0, sizeof(desc));
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
 
     /* It should come back as a bunch of empty strings */
     desc.description="";
@@ -648,7 +647,7 @@ static void test_load_save(void)
     desc.icon="c:\\nonexistent\\icon\\file";
     desc.icon_id=1234;
     desc.hotkey=0;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     check_lnk(lnkfile, &desc, 0x0);
 
     r=GetModuleFileNameA(NULL, mypath, sizeof(mypath));
@@ -672,7 +671,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     check_lnk(lnkfile, &desc, 0x0);
 
     /* Test omitting .exe from an absolute path */
@@ -689,7 +688,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     strcat(realpath, ".exe");
     check_lnk(lnkfile, &desc, 0x4);
 
@@ -703,7 +702,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     /* Check that link is created to proper location */
     SearchPathA( NULL, desc.path, NULL, MAX_PATH, realpath, NULL);
     desc.path=realpath;
@@ -719,7 +718,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     /* Check that link is created to proper location */
     SearchPathA( NULL, "rundll32", NULL, MAX_PATH, realpath, NULL);
     desc.path=realpath;
@@ -750,7 +749,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     check_lnk(lnkfile, &desc, 0x0);
 
     r = GetShortPathNameA(mydir, mypath, sizeof(mypath));
@@ -770,7 +769,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     desc.path=realpath;
     check_lnk(lnkfile, &desc, 0x0);
 
@@ -800,7 +799,7 @@ static void test_load_save(void)
     desc.icon=mypath;
     desc.icon_id=0;
     desc.hotkey=0x1234;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
     desc.path = realpath;
     check_lnk(lnkfile, &desc, 0x4);
 
diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c
index 8ee662321d1..2481fb6faa2 100644
--- a/dlls/shell32/tests/shlexec.c
+++ b/dlls/shell32/tests/shlexec.c
@@ -2742,7 +2742,7 @@ static void init_test(void)
     desc.icon=NULL;
     desc.icon_id=0;
     desc.hotkey=0;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
 
     sprintf(filename, "%s\\test_shortcut_exe.lnk", tmpdir);
     MultiByteToWideChar(CP_ACP, 0, filename, -1, lnkfile, ARRAY_SIZE(lnkfile));
@@ -2756,7 +2756,7 @@ static void init_test(void)
     desc.icon=NULL;
     desc.icon_id=0;
     desc.hotkey=0;
-    create_lnk(lnkfile, &desc, 0);
+    create_lnk(lnkfile, &desc);
 
     /* Create a basic association suitable for most tests */
     if (!create_test_association(".shlexec"))




More information about the wine-cvs mailing list